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

Update file parsing to handle lines with only whitespace. #24

Closed
wants to merge 1 commit into from

Conversation

captainreality
Copy link

This commit updates file parsing so that lines that contain solely whitespace (as defined by by Unicode) are ignored. It's fairly common for people to accidentally create CODEOWNERS files with lines containing solely whitespace, and as it's semantically meaningless, it's better to ignore them then raise an error.

This commit updates file parsing so that lines that contain solely
whitespace (as defined by by Unicode) are ignored. It's fairly common
for people to accidentally create CODEOWNERS files with lines containing
solely whitespace, and as it's semantically meaningless, it's better to
ignore them then raise an error.
This was referenced May 29, 2024
@captainreality
Copy link
Author

Closed the PR because it was merged in #26.

@hmarr
Copy link
Owner

hmarr commented Jun 26, 2024

Thanks @captainreality, I've just pushed a fix for this in 06d1caa

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