Skip to content

Conversation

@MrJoy
Copy link
Contributor

@MrJoy MrJoy commented May 3, 2022

Because rbenv uses shims, the existence of foreman in any gemset will result in this test passing (indicating that the tool is available), whether it's in the current gemset or not.

This test, courtesy of @t27duck, ensures that foreman is actually operable in the current context.

This commit mirrors the identical commit in cssbundling-rails:

rails/cssbundling-rails@dff8845

Because rbenv uses shims, the existence of foreman in any gemset will result in this test passing (indicating that the tool is available), whether it's in the current gemset or not.

This test, courtesy of @t27duck, ensures that foreman is actually operable in the current context.

This commit mirrors the identical commit in cssbundling-rails:

rails/cssbundling-rails@dff8845
@dhh dhh merged commit 90833fb into rails:main May 3, 2022
@MrJoy MrJoy deleted the fix-bin-dev-rbenv-compatibility branch May 3, 2022 19: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.

2 participants