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

Support '"' to identify heredoc delimiters in Shell lexer #1411

Merged
merged 1 commit into from Jan 24, 2020

Conversation

pyrmont
Copy link
Contributor

@pyrmont pyrmont commented Jan 24, 2020

Heredoc delimiters in the Shell lexer can be identified by the use of '. This PR adds support for the use of ".

This fixes #1409.

@pyrmont pyrmont added the needs-review The PR needs to be reviewed label Jan 24, 2020
@pyrmont pyrmont self-assigned this Jan 24, 2020
@pyrmont
Copy link
Contributor Author

pyrmont commented Jan 24, 2020

This looks like it fixes the problem on my end, @Cartoonman. How's it look to you?

@Cartoonman
Copy link

Tried your patch on my local instance and LGTM. Thanks @pyrmont!

@pyrmont pyrmont merged commit 25da9ab into rouge-ruby:master Jan 24, 2020
@pyrmont
Copy link
Contributor Author

pyrmont commented Jan 24, 2020

@Cartoonman Sweet! Thanks for the report :) We release new versions of Rouge on a monthly cadence at the moment. Version 3.16.0 will have this change incorporated and is scheduled to be pushed out on Tuesday 11 February.

@pyrmont pyrmont deleted the bugfix.shell-heredoc-delimiters branch January 24, 2020 23:36
@pyrmont pyrmont removed the needs-review The PR needs to be reviewed label Jan 24, 2020
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.

Heredoc with double quoted delimiter not properly handled
2 participants