Skip to content

Conversation

filipesperandio
Copy link
Contributor

No description provided.

@filipesperandio filipesperandio requested review from codeclimate-hermes and removed request for codeclimate-hermes June 5, 2019 16:43
bin/newver Outdated
git checkout -b "newver/${NEWVER}"

ed Gemfile <<-EDITS
/gem "parser",/c
/gem "parser"/c
Copy link
Contributor

Choose a reason for hiding this comment

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

can I be honest? I have no idea how to use ed or what this change accomplishes. Would you mind providing some context?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

hahaha struggled with this a bit too, never fully learned ed...
On master, this line is:
gem "parser", "~> 2.4.0"
But recent channels it is:
gem "parser"
The change make sure on either case the result is just gem "parser"... which could actually go away since it accomplishes nothing in the new approach for forking from latest channel... just realizing now that I am explaining it.

bin/newver Outdated
previous_channel() {
ruby -e "version = \"${NEWVER}\".match(/(\d+).(\d+).(\d+)/); \
minor = version[2].to_i - 1; \
puts \"channel/rubocop-#{version[1]}-#{minor}\""
Copy link
Contributor

Choose a reason for hiding this comment

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

This is very clever. IMO it wouldn't be crazy to rewrite this whole script in ruby with a #!/usr/bin/env ruby shebang if that will make this all easier

Happy to stick with this, though. It might make your life a little easier to do something like:

cat <<EOM | ruby
  ruby goes here without need of escaping quotes
EOM

@filipesperandio filipesperandio merged commit 75eccb1 into master Jun 7, 2019
@filipesperandio filipesperandio deleted the fe/newver-off-recent-channel branch June 7, 2019 09:50
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