Skip to content
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

Improve spellcheck script to allow for projects to specify additional sources #1100

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

adeljo-amd
Copy link

This PR contains the following update:

  • Added config to spell-check C++ files in .spellcheck.yaml
  • Added a file .spellcheck.local.yaml that can be used as a reference for other projects when they want to specify additional sources
  • Added a python script docs/yaml_merger.py to merge any project's .spellcheck.local.yaml with the main .spellcheck.yaml in this repository. The script simply checks the nodes name and source and ensures they match before merging the additional sources (node depth must match too). Anything else is ignored.

@adeljo-amd adeljo-amd requested a review from neon60 January 20, 2025 11:15
@adeljo-amd adeljo-amd self-assigned this Jan 20, 2025
@adeljo-amd adeljo-amd requested a review from a team as a code owner January 20, 2025 11:15
@adeljo-amd adeljo-amd force-pushed the local_spellcheck branch 4 times, most recently from 4a04c4b to 8a5852a Compare January 20, 2025 11:42
@adeljo-amd adeljo-amd changed the title Improve spellcheck script to allow for projects to specify additional sources Draft: Improve spellcheck script to allow for projects to specify additional sources Jan 20, 2025
@adeljo-amd adeljo-amd changed the title Draft: Improve spellcheck script to allow for projects to specify additional sources Improve spellcheck script to allow for projects to specify additional sources Jan 20, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant