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

Trim trailing whitespace in config file #342

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

Conversation

jacob-horton
Copy link

I noticed one of the options in my config file wasn't working. It turned out this is because it had a space at the end of the line

I thought it could be nice to trim the whitespace at the end of the line so we don't leave people head scratching like I was! If preferred, we could spit out a warning instead of trimming. Then that would make it easier to find out why a configuration isn't working

I don't often write C, so feel free to suggest any changes, I'll be happy to implement them!

Also, I'm new to this repo, so if there's already something like trim_end I should be using instead, or if I should move the function to another file, please let me know 🙂

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

1 participant