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

modulesync 2.5.1 and drop Puppet 4 #282

Merged
merged 6 commits into from
Mar 8, 2019
Merged

modulesync 2.5.1 and drop Puppet 4 #282

merged 6 commits into from
Mar 8, 2019

Conversation

bastelfreak
Copy link
Member

modulesync 2.5.1

@ekohl
Copy link
Member

ekohl commented Feb 28, 2019

Now has a merge conflict

spec/defines/selinux_exec_restorecon_spec.rb Outdated Show resolved Hide resolved
spec/default_module_facts.yml Outdated Show resolved Hide resolved
manifests/params.pp Outdated Show resolved Hide resolved
@bastelfreak bastelfreak force-pushed the modulesync branch 3 times, most recently from ef68ff2 to 5d1034a Compare February 28, 2019 21:34
Copy link
Member

@ekohl ekohl left a comment

Choose a reason for hiding this comment

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

This does make it a major version bump. I generally approve but how about we do a last release of all the current stuff before we do a major?

@@ -9,7 +9,8 @@
osfamily: 'RedHat',
Copy link
Member

Choose a reason for hiding this comment

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

You can drop the old facts here if you want.

manifests/params.pp Show resolved Hide resolved
@bastelfreak
Copy link
Member Author

@ekohl IMO merging this and doing a major release is fine, that's wha we agreed on in voxpupuli/voxpupuli.github.io#157

@ekohl
Copy link
Member

ekohl commented Mar 1, 2019

@bastelfreak I'd prefer to do a release, bump major and then wait for other changes to come in

@dhollinger
Copy link
Member

Based on our conversation in voxpupuli/voxpupuli.github.io#157, we wanted to not ever do a release just for removing puppet 4. As such, I'm ok with this PR, @ekohl. Doing a minor/patch release to prepare for a major release change that removes Puppet 4 is exactly what the community agreed to avoid doing.

@bastelfreak bastelfreak merged commit a43c5cd into master Mar 8, 2019
@bastelfreak bastelfreak deleted the modulesync branch March 8, 2019 19:05
EmRowlands pushed a commit to EmRowlands/puppet-selinux that referenced this pull request Mar 29, 2023
modulesync 2.5.1 and drop Puppet 4
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants