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

Fix codespell #201

Merged
merged 1 commit into from
Aug 23, 2022
Merged

Fix codespell #201

merged 1 commit into from
Aug 23, 2022

Conversation

chrisdoherty4
Copy link
Member

@chrisdoherty4 chrisdoherty4 commented Aug 22, 2022

Fix CodeSpell that fails on a false positive. Its not clear why this began but my hunch is that the @master was playing a part. v1.0 was released 4 days ago.

@chrisdoherty4 chrisdoherty4 marked this pull request as ready for review August 22, 2022 19:13
Signed-off-by: Chris Doherty <chris.doherty4@gmail.com>
codespell:
name: Codespell
runs-on: ubuntu-latest
steps:
- name: Checkout Code
uses: actions/checkout@v3
- name: Codespell
uses: codespell-project/actions-codespell@master
uses: codespell-project/actions-codespell@v1.0
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tag at v1.0

with:
check_filenames: true
check_hidden: true
skip: './.git,./go.mod,./go.sum'
skip: "./.git,./go.mod,./go.sum"
ignore_words_list: ro
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Add ignore words list with ro to ignore the common read-only abbreviation.

@displague displague added the ready-to-merge Signal to Mergify to merge the PR. label Aug 22, 2022
@chrisdoherty4 chrisdoherty4 removed the request for review from micahhausler August 23, 2022 14:58
@mergify mergify bot merged commit b99c859 into tinkerbell:main Aug 23, 2022
@displague displague added this to the v0.2 milestone Aug 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready-to-merge Signal to Mergify to merge the PR.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants