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

Pin string and charlist to opening " or ', so that quote handler can use it for auto-insert closing #741

Merged
merged 2 commits into from Jul 21, 2017

Conversation

KronicDeth
Copy link
Owner

Fixes #582

Changelog

Enhancements

  • Instead of pinning position (1 for lines or 3 for heredocs), pin *_PROMOTER token, so it's more obvious the pattern is that the promoter is pinned.

Bug Fixes

  • Pin promoter for all quotes, including the previously missing stringLine and charListLine.
  • Only insert closing quote if previous token was an opening quote.

Heredoc promoters were pinned, but line promoters weren't.  Instead of
pinning position, pin *_PROMOTER token, so it's more obvious the pattern
is that the promoter is pinned.
@KronicDeth KronicDeth self-assigned this Jul 21, 2017
@KronicDeth KronicDeth merged commit 5c5ec86 into master Jul 21, 2017
@KronicDeth KronicDeth added this to the v6.0.0 milestone Jul 21, 2017
@KronicDeth KronicDeth deleted the 582 branch July 21, 2017 13:47
KronicDeth added a commit that referenced this pull request Jul 21, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant