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: residual space after section causes bad parsing #203

Closed
wants to merge 2 commits into from

Conversation

wraithgar
Copy link
Member

Any character left on a line after a section should be ignored.

Any character left on a line after a section should be ignored.
@wraithgar wraithgar requested a review from a team as a code owner April 12, 2023 01:34
@wraithgar wraithgar requested a review from nlf April 12, 2023 01:34
@wraithgar
Copy link
Member Author

Rebase/recreation of #123 to let CI run.

lib/ini.js Outdated Show resolved Hide resolved
lib/ini.js Outdated Show resolved Hide resolved
@Nautigsam
Copy link
Contributor

Rebase/recreation of #123 to let CI run.

Hi, I can rebase my branch if you prefer.

@wraithgar
Copy link
Member Author

@Nautigsam oh by all means! Most of the PRs were old enough that I didn't want to hassle the authors about it anymore.

Please note the \s change to the regex.

@wraithgar wraithgar closed this Apr 12, 2023
@wraithgar wraithgar deleted the gar/pr123 branch April 12, 2023 14:30
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.

None yet

2 participants