Skip to content

Commit

Permalink
Merge pull request #958 from heroku/schneems/fix-logic-inversion
Browse files Browse the repository at this point in the history
Fix version download warning logic
  • Loading branch information
schneems authored Mar 6, 2020
2 parents da3fdae + 2e84707 commit 3f28116
Show file tree
Hide file tree
Showing 4 changed files with 8 additions and 2 deletions.
5 changes: 5 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
@@ -1,5 +1,10 @@
## Master (unreleased)


## v210 (3/6/2020)

* Fix version download error warning inversion logic (https://github.com/heroku/heroku-buildpack-ruby/pull/958)

## v209 (3/5/2020)

* Fix bug in version download error message logic (https://github.com/heroku/heroku-buildpack-ruby/pull/957)
Expand Down
2 changes: 1 addition & 1 deletion lib/language_pack/ruby.rb
Original file line number Diff line number Diff line change
Expand Up @@ -450,7 +450,7 @@ def warn_outdated_ruby
def warn_stack_upgrade
return unless defined?(@ruby_download_check)
return unless @ruby_download_check.next_stack(current_stack: stack)
return unless @ruby_download_check.exists_on_next_stack?(current_stack: stack)
return if @ruby_download_check.exists_on_next_stack?(current_stack: stack)

warn(<<~WARNING)
Your Ruby version is not present on the next stack
Expand Down
2 changes: 1 addition & 1 deletion lib/language_pack/version.rb
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,6 @@

module LanguagePack
class LanguagePack::Base
BUILDPACK_VERSION = "v209"
BUILDPACK_VERSION = "v210"
end
end
1 change: 1 addition & 0 deletions spec/hatchet/ruby_spec.rb
Original file line number Diff line number Diff line change
Expand Up @@ -100,6 +100,7 @@
Hatchet::Runner.new("default_ruby").deploy do |app, heroku|
expect(app.output).to include("Writing config/database.yml to read from DATABASE_URL")
expect(app.output).not_to include("Your app was upgraded to bundler")
expect(app.output).not_to include("Your Ruby version is not present on the next stack")
end
end
end
Expand Down

0 comments on commit 3f28116

Please sign in to comment.