New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Check spelling should ignore language keywords based on file type #2
Comments
Maybe we could have a directory for allowed words, then each of the file could look like this:
|
Working approach todayTo define multiple language sets, one can:
name: Spell checking
on:
push:
branches:
- "**"
tags-ignore:
- "**"
schedule:
# * is a special character in YAML so you have to quote this string
- cron: '5 * * * *'
jobs:
spell-check:
name: Spell checker
runs-on: ubuntu-latest
continue-on-error: true
strategy:
matrix:
language: [python, java]
steps:
- uses: actions/checkout@v2.0.0
with:
fetch-depth: 2
- uses: check-spelling/check-spelling@0.0.16-alpha
env:
bucket: .github/actions/spelling
project: ${{ matrix.language }}
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
The downside to this is that you will get distinct reports (i.e. potentially multiple comments) for spelling errors from different kinds of files. -- I'd love to get some implementation experience here. I made a simple version of the above as a Proof of Concept. It does work. The next step would be properly setting up dictionaries for the programming languages... It is possible for an action to pass data off to another stage, so it might be possible for me to add support for coalescing reports. If one wanted to be truly adventurous, the workflow file allows scoping jobs to changed files. ThoughtsOrganization common configGitHub itself has the concept of a magical per organization .github repository. Per organization shared rulesBecause of the way configurations work for this action, you could just add an extra Support for
|
I've promoted the preceding comment to the wiki as Configuration: Multiple programming languages. The features for this are all in 0.0.16-alpha. (It's silly for me to repeatedly update a comment in an issue ....) The rest of the request should be addressed by Feature: Area dictionaries which I'm starting to flesh out -- I've created some of the basis for it in 0.0.17-alpha (which I hope to release real-soon-now), I think the initial area dictionaries will come within the next couple of releases. Some disclaimers:
|
No description provided.
The text was updated successfully, but these errors were encountered: