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
use heredocs #2197
base: master
Are you sure you want to change the base?
use heredocs #2197
Conversation
Heredocs makes it a bit nicer to read and write inline text.
puts " cd \"#{target}\"; find -type f -not -name sha1sums -print0 | \\" | ||
puts" xargs -0 sha1sum -b >sha1sums" | ||
puts "(or use the --force switch)" | ||
print <<-EOF.gsub(/^ *\| /, '') |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Style/StringLiterals: Prefer double-quoted strings unless you need single quotes to avoid extra backslashes for escaping. (https://github.com/bbatsov/ruby-style-guide#consistent-string-literals)
Codeclimate works, but your changes are getting removed anyway within #2195, so closing this issue. |
@tboerger but now Jenna can't see that it worked ... |
Reopened just to show the Codeclimate support that it works without a fork. |
@aspiers can this be closed now? |
@aspiers Is this still an issue you want to keep open? |
@nicolasbock Sure why not - all those |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's merge this!
@aspiers needs rebase |
Heredocs makes it a bit nicer to read and write inline text.
This is a half-serious untested pull-request submitted mainly to test whether CodeClimate works on PRs within a single repo (in contrast we know for sure that there are existing issues with cross-fork PRs, but CodeClimate said they should have these fixed very soon).