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

Fix heredoc within parens formatting causing subsequent syntax error #116

Merged
merged 1 commit into from
Jul 27, 2018

Conversation

mjago
Copy link
Member

@mjago mjago commented Jul 21, 2018

Fix for issue #88

@mjago
Copy link
Member Author

mjago commented Jul 26, 2018

@gingermusketeer @splattael @danielma Any input on this PR guys? This is a fix for a reported bug that leaves code broken so I think it should be considered.

I will probably focus on fixing bugs on the new formatter moving forward - but I think it was important to fix these breaking bugs on current formatter. Style issues on current formatter are less urgent than getting new formatter production (beta) ready. Also, Ripper changes need to be followed up to ensure we know what is coming around the corner 😄

CHANGELOG.md Outdated
@@ -7,6 +7,7 @@ and this project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.
## [Unreleased]

### Fixed
- Fix `quote_style` Bug: Rufo breaks HEREDOC syntax. (issue [#88](https://github.com/ruby-formatter/rufo/issues/88)). This fixes heredoc within parens formatting causing bad formatting and subsequent syntax error.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What does this change have to do with quote_style? Was the bug introduced by the change that added it?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oops ~ I thought quote_style was some fancy markdown formatting that was being used in the changelog (cut-paste error) 😊 I'll fix that

@gingermusketeer
Copy link
Member

@mjago Getting this sorted is great. Nice for folks to be able to use this soon! Perhaps we should do a release after this is merged?

Copy link
Member

@splattael splattael left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💚

@mjago mjago merged commit c1ae045 into ruby-formatter:master Jul 27, 2018
@mjago
Copy link
Member Author

mjago commented Jul 27, 2018

@splattael do you wanna do a release?

@mjago mjago deleted the heredoc_error branch July 27, 2018 06:17
@gingermusketeer
Copy link
Member

@mjago All your changes have been released with v0.4.0 :)

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.

3 participants