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

Add pessimistic constraint to puppet_forge dependency #297

Merged
merged 1 commit into from May 6, 2015

Conversation

scotje
Copy link
Contributor

@scotje scotje commented May 4, 2015

At some point there will probably be a 2.x release of the puppet_forge gem that may not be backwards compatible. Pessimistic version constraints on dependencies is also a RubyGems best practice: http://guides.rubygems.org/patterns/#declaring-dependencies

@carlossg
Copy link
Collaborator

carlossg commented May 5, 2015

I'm more inclined to leave it as is and if that is the case in the future users can explicitly add the constraint themselves. I've seen too many cases where a constraint set early screws up downstream users that want to use a newer version

@scotje
Copy link
Contributor Author

scotje commented May 5, 2015

The proposed constraint allows everything < 2.0.0. In the current scenario, if we push a puppet_forge 2.x, anyone who has not independently constrained on puppet_forge 1.x is likely going to break their install the next time they gem update or bundle update.

If it turns out that existing versions of librarian-puppet are compatible with puppet_forge 2.x, it seems simple enough to publish a z-release with an expanded dependency range so people can upgrade.

Obviously it's up to you guys what you want to do. Consider this an early heads up that a 2.x puppet_forge is looking pretty likely at this point and it will almost certainly not be fully backwards compatible with the 1.x series.

@carlossg
Copy link
Collaborator

carlossg commented May 6, 2015

I misunderstood, if you already know that is not backwards compatible then it makes sense

carlossg added a commit that referenced this pull request May 6, 2015
Add pessimistic constraint to puppet_forge dependency
@carlossg carlossg merged commit 8a8b694 into rodjek:master May 6, 2015
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

2 participants