Skip to content

Conversation

@marcparadise
Copy link
Member

Ruby 2.5 and earlier are all EOL and are no longer receiving security updates.

Signed-off-by: Marc A. Paradise marc.paradise@gmail.com

@marcparadise
Copy link
Member Author

Note that windows build was failing prior to this PR, and is not addressed here.

@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@marcparadise
Copy link
Member Author

Will update this further - even though we can't test these older versions, there may still be cases where they get pulled in and will work (like chef-install cookbook for upgrades). After update, it will leave in 'soft' support (conditional gems by version) but remove the unsupported ruby versions.

@jaymzh jaymzh added the Status: Waiting on Contributor A pull request that has unresolved requested actions from the author. label Nov 1, 2022
@jaymzh
Copy link
Collaborator

jaymzh commented Nov 1, 2022

@marcparadise ping? :)

@tpowell-progress tpowell-progress self-assigned this Apr 25, 2023
Ruby 2.5 and earlier are all EOL and are no longer receiving security
updates.

Signed-off-by: Marc A. Paradise <marc.paradise@gmail.com>
pry-byebug is not supported on all platforms, and is also not necessary
for running pipeline tests. Excluding gem group 'debug' from tests prevents
failures related to unsupported platforms.

Signed-off-by: Marc A. Paradise <marc.paradise@gmail.com>
Signed-off-by: Thomas Powell <thomas.powell@progress.com>
@tpowell-progress tpowell-progress requested a review from a team as a code owner October 31, 2023 17:49
@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@jaymzh
Copy link
Collaborator

jaymzh commented Oct 29, 2024

This relates to: #400

@marcparadise - I'm unclear on the status of this PR - I think we can merge, but it's been 2 years, so we should at least rebase.

@sonarqubecloud
Copy link

spec.executables = ["mixlib-install"]
spec.require_paths = ["lib"]

spec.required_ruby_version = ">= 2.6.0"
Copy link
Collaborator

Choose a reason for hiding this comment

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

I think we decided not to force newer versions so people can use this on like an old Mac to get the latest chef... we wanted to go "soft" min requirement.

gem "webrick"
gem "webmock", "~> 3.4"
gem "contracts", "~> 0.16.0" # this entry can go away when ruby < 3 support is gone
if RUBY_VERSION < "2.6.0"
Copy link
Collaborator

Choose a reason for hiding this comment

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

probably wanna leave the conditional here....


gemspec

gem "chef-utils", "= 16.6.14" if RUBY_VERSION < "2.6.0"
Copy link
Collaborator

Choose a reason for hiding this comment

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

shall we leave this as well?

@sonarqubecloud
Copy link

@johnmccrae
Copy link

@marcparadise Is this still needed?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Status: Waiting on Contributor A pull request that has unresolved requested actions from the author.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants