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

(fix) Set rubocop ruby version to 2.4 #360

Merged
merged 2 commits into from
Nov 30, 2020

Conversation

tuxmea
Copy link
Contributor

@tuxmea tuxmea commented Nov 12, 2020

2.1 has some changes to hash syntax.
using 2.1 shows false positives

2.1 has some changes to hash syntax.
using 2.1 shows false positives
@tuxmea tuxmea requested review from a team as code owners November 12, 2020 12:44
Copy link
Contributor

@DavidS DavidS left a comment

Choose a reason for hiding this comment

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

Hi @tuxmea ,

  1. as it is this seems to be breaking pdk validate
  2. as long as we have to support puppet4/ruby2.1 this will create incompatible code or failing validations

@DavidS DavidS self-assigned this Nov 16, 2020
@tuxmea
Copy link
Contributor Author

tuxmea commented Nov 16, 2020

rubcop complained about wrong ruby version, only up to 2.4 is accepted. are there already any deadlines for dropping puppet 4 / ruby 2.1 support?
Puppet 7 seems to be already available at nightly builds.

@tuxmea tuxmea requested a review from DavidS November 20, 2020 09:00
@tuxmea tuxmea changed the title set rubocop ruby version to 2.5 set rubocop ruby version to 2.4 Nov 23, 2020
@tuxmea
Copy link
Contributor Author

tuxmea commented Nov 23, 2020

Hi @tuxmea ,

1. as it is this seems to be breaking `pdk validate`

Fixed.

2. as long as we have to support puppet4/ruby2.1 this will create incompatible code or failing validations

Support Puppet 4? Are you sure?

@DavidS
Copy link
Contributor

DavidS commented Nov 23, 2020

puppetlabs/pdk-vanagon#268 is currently in progress to remove puppet4 and ruby 2.1 from pdk. That'll unblock this work.

@DavidS
Copy link
Contributor

DavidS commented Nov 27, 2020

the pdk change got merged, so this is unblocked. Since I'm away next week, I'll leave it to the team to merge on Community Monday.

@sanfrancrisko sanfrancrisko changed the title set rubocop ruby version to 2.4 (fix) Set rubocop ruby version to 2.4 Nov 30, 2020
@sanfrancrisko sanfrancrisko merged commit 62126e1 into puppetlabs:main Nov 30, 2020
@tuxmea tuxmea deleted the alfkem_rubocop_ruby_version branch November 30, 2020 11:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants