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

Improve disambiguation rules on .pp extension #1898

Merged
merged 1 commit into from
Dec 12, 2022

Conversation

tancnle
Copy link
Collaborator

@tancnle tancnle commented Dec 4, 2022

This tightens the disambiguation around .pp extension that is shared between Puppet and Pascal lexer.

  • Add qualified names check for Puppet
  • Ensure Pascal keywords are at the start of strings

Closes #1866

This tightens the disambiguation around `.pp` extension that is shared
between Puppet and Pascal lexer.

- Add qualified names check for Puppet
- Ensure Pascal keywords are at the start of strings
@tancnle
Copy link
Collaborator Author

tancnle commented Dec 4, 2022

👋🏼 @jamespwilliams @Alexey-T keen to hear your thoughts on this change 🙏🏼

@tancnle tancnle merged commit 608d060 into rouge-ruby:master Dec 12, 2022
@tancnle tancnle deleted the fix/puppet-pascal-disambiguation branch December 12, 2022 00:37
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.

Pascal detection rules are too keen
1 participant