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 comments reggression #18

Merged
merged 1 commit into from
Sep 19, 2022
Merged

Fix comments reggression #18

merged 1 commit into from
Sep 19, 2022

Conversation

BlackDex
Copy link

@BlackDex BlackDex commented Sep 5, 2022

This PR fixes #12

Added a WhiteSpace ParseState and check for a comment after a
whitespace. Also return the current position of the iter to correctly
strip the comments of from the line to prevent quoted comments or
non-whitespace surrounded hash chars.

Also updated the tests to match these cases.
And added an example code to easily show the output.

This PR fixes allan2#12

Added a WhiteSpace ParseState and check for a comment after a
whitespace. Also return the current position of the iter to correctly
strip the comments of from the line to prevent quoted comments or
non-whitespace surrounded hash chars.

Also updated the tests to match these cases.
And added an example code to easily show the output.
@allan2 allan2 merged commit 39ab2e6 into allan2:master Sep 19, 2022
allan2 pushed a commit that referenced this pull request Sep 19, 2022
 Another fix for multiline support [(PR #18)](#18). Thanks [BlackDex](https://github.com/BlackDex) and [LeoniePhiline](https://github.com/LeoniePhiline)
allan2 pushed a commit that referenced this pull request Sep 19, 2022
Minimum Supported Rust Version is now 1.58.1. [(PR #18)](#21) by [rillian](https://github.com/rillian)
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.

Regression: Multiline support breaks .env files with quotes in comments
2 participants