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

(maint) stop restricting bundler and ruby in Gemfile #3242

Merged
merged 1 commit into from
Oct 5, 2023

Conversation

donoghuc
Copy link
Member

@donoghuc donoghuc commented Oct 5, 2023

Bolt's gemspec allows ruby 2.5 and up. This is the source of truth for ruby compatability. This commit removes the versions in the Gemfile. The restriction was added to try to get rid of a warning message when using a bundler version that is too old. This is something bundler users can manage themselves if the warning is annoying them.

!bug

  • Stop restricting ruby version in Gemfile

    The source of truth for minimum ruby version is in the gemspec. Stop duplicating that and risking conflicts in the Gemfile.

Bolt's gemspec allows ruby 2.5 and up. This is the source of truth for ruby compatability. This commit removes the versions in the Gemfile. The restriction was added to try to get rid of a warning message when using a bundler version that is too old. This is something bundler users can manage themselves if the warning is annoying them.

!bug

* **Stop restricting ruby version in Gemfile**

  The source of truth for minimum ruby version is in the gemspec. Stop duplicating
  that and risking conflicts in the Gemfile.
@donoghuc donoghuc requested a review from a team as a code owner October 5, 2023 21:06
@mcdonaldseanp mcdonaldseanp merged commit 6fb8a0a into puppetlabs:main Oct 5, 2023
42 checks passed
Copy link
Contributor

@tlehman tlehman left a comment

Choose a reason for hiding this comment

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

LGTM

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.

None yet

3 participants