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: GitHub Action to scan correct folders for typos #34

Merged
merged 1 commit into from
Jan 20, 2025

Conversation

ellenfieldn
Copy link
Contributor

I noticed in #33 that the check-spelling action was failing due to attempting to scan a folder that doesn't exist.
image

The pages folder doesn't exist and causes the typos command to fail, so I just removed it.

I validated locally, but saw a slightly different error message. It's possible there is another issue as well, but this should at least fix something.
image

@ellenfieldn
Copy link
Contributor Author

QQ: Should I also be creating issues for this type of thing before submitting the PR?

@NiHaiden
Copy link
Owner

QQ: Should I also be creating issues for this type of thing before submitting the PR?

If you identify something that only I can fix or similar then yeah create an issue, otherwise not really needed ^^

@NiHaiden
Copy link
Owner

Also thank you for your PRs ❤️

@NiHaiden NiHaiden merged commit b51bf1e into NiHaiden:main Jan 20, 2025
1 check passed
@ellenfieldn ellenfieldn deleted the fix-spellcheck branch January 25, 2025 01:16
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.

2 participants