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

Bugfix: Backslash stripped in string literal #13

Merged
merged 5 commits into from
Jan 12, 2020

Conversation

twbartel
Copy link
Collaborator

@twbartel twbartel commented Jan 3, 2020

This PR aims to fix issue #11, where backslashes are dropped from string literals. It has to wait for a Melody release.

…ng-literal

* master: (35 commits)
  Remove duplicate code/snapshot
  Update snapshots
  Get range ignore to work Add test cases
  No spaces around ".." operator
  Optionally remove spaces around filter pipe
  Upgrade to Melody 1.6.0
  Better handling of empty FOR statements
  For PrintExpressionStatement, condense certain node types This was done before for VariableDeclaration already
  No space in empty object literals
  Add failing test case (Twig comment with whitespace control) that should be fixed with the next Melody release
  Print empty IF statements inline
  Add test cases and snapshots for Twig coding standards
  Rebasing on master
  Release v0.1.1
  Add simpler handling for zero-argument call expressions
  No spaces around ".." operator
  Optionally remove spaces around filter pipe
  Upgrade to Melody 1.6.0
  Better handling of empty FOR statements
  For PrintExpressionStatement, condense certain node types This was done before for VariableDeclaration already
  ...
Relocate test case to final location
@twbartel twbartel changed the title [Waiting for Melody release] Bugfix: Backslash stripped in string literal Bugfix: Backslash stripped in string literal Jan 12, 2020
@twbartel twbartel merged commit 3dd3e9c into master Jan 12, 2020
@twbartel twbartel deleted the bugfix/issue-11-backslash-stripped-in-string-literal branch January 12, 2020 00:08
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.

1 participant