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

Combine import and package rules in Java lexer #1389

Merged
merged 1 commit into from Jan 17, 2020

Conversation

pyrmont
Copy link
Contributor

@pyrmont pyrmont commented Jan 14, 2020

The lexing that is performed for packages imported using the import keyword should be the same for the package keyword. This is consistent with the current behaviour in the Pygments lexer.

This fixes #1388.

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

pyrmont commented Jan 14, 2020

@mojavelinux This looks good to me but let me know if you see anything awry.

@mojavelinux
Copy link
Contributor

Looks good to me!

@pyrmont pyrmont merged commit c999334 into rouge-ruby:master Jan 17, 2020
@pyrmont pyrmont deleted the bugfix.java-package-names branch January 17, 2020 18:52
@pyrmont
Copy link
Contributor Author

pyrmont commented Jan 17, 2020

Great! Will be in the next version :)

@mojavelinux
Copy link
Contributor

Thanks @pyrmont! This will really help our large Java community in the Asciidoctor ecosystem, since they have a lot of these ;)

@pyrmont pyrmont removed the needs-review The PR needs to be reviewed label Jan 17, 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.

First segment of Java package namespace is not highlighted
2 participants