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

Remove version pin for rainbow dev dependency #553

Merged
merged 2 commits into from
Feb 3, 2025

Conversation

mattbrictson
Copy link
Member

The rainbow gem is a dependency of rubocop, which we use as a development dependency on this project. Previously, we were explicitly declaring rainbow as a dependency so that we could pin it at a specific version. This was done to ensure the development gems could be installed on old versions of Ruby in CI (see f57347a).

Now that sshkit no longer supports Ruby < 2.5, this workaround is no longer needed. This commit removes the explicit rainbow dev dependency.

The `rainbow` gem is a dependency of rubocop, which we use as a
development dependency on this project. Previously, we were explicitly
declaring rainbow as a dependency so that we could pin it at a specific
version. This was done to ensure the development gems could be installed
on old versions of Ruby in CI (see f57347a).

Now that sshkit no longer supports Ruby < 2.5, this workaround is no
longer needed. This commit removes the explicit rainbow dev dependency.
@mattbrictson mattbrictson added the 🏠 Housekeeping Non-user facing cleanup and maintenance label Feb 3, 2025
@mattbrictson mattbrictson enabled auto-merge (squash) February 3, 2025 15:58
@mattbrictson mattbrictson merged commit 495ea1d into master Feb 3, 2025
15 checks passed
@mattbrictson mattbrictson deleted the chores/un-pin-rainbow branch February 3, 2025 15:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🏠 Housekeeping Non-user facing cleanup and maintenance
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants