Skip to content
This repository has been archived by the owner on Apr 14, 2021. It is now read-only.

Merge GemVersionPromoter into outdated cmd #5058

Closed
wants to merge 2 commits into from

Conversation

ryanfox1985
Copy link
Contributor

Hi I created this PR according with:
#4841

Regards.

@chrismo
Copy link
Contributor

chrismo commented Oct 6, 2016

thx - i actually had started working on this locally, i'm not sure some of these changes are going to be necessary, but we'll see -- here's my new PR in progress now: #5061.

@ryanfox1985
Copy link
Contributor Author

ryanfox1985 commented Oct 6, 2016

@chrismo, If you want I can apply your renames of the input variables [patch, minor, major].

@ryanfox1985
Copy link
Contributor Author

@chrismo, can you do the review here?
I can explain the code.

Regards.

@homu
Copy link
Contributor

homu commented Oct 22, 2016

☔ The latest upstream changes (presumably #5061) made this pull request unmergeable. Please resolve the merge conflicts.

@ryanfox1985
Copy link
Contributor Author

I think that I need to fix one test and rollback the changes on definition.rb (because I don't need).

@chrismo
Copy link
Contributor

chrismo commented Oct 22, 2016

@ryanfox1985 I ended up taking care of this in #5061, unless you see something there I missed, we can close yours.

end

if options["filter-major"] || options["filter-minor"] || options["filter-patch"]
active_spec = Bundler.definition.gem_version_promoter.sort_versions(dependency, spec_groups).last
Copy link
Contributor Author

@ryanfox1985 ryanfox1985 Oct 23, 2016

Choose a reason for hiding this comment

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

@chrismo In master code is not calling to gem_version_promoter.sort_versions. I think it's a good idea keep calling to gem_version_promoter (what do you think?)

I created another PR #5114 for clean code stuff (I will remove the refactors from this PR).

@bundlerbot
Copy link
Collaborator

☔ The latest upstream changes (presumably #5176) made this pull request unmergeable. Please resolve the merge conflicts.

1 similar comment
@homu
Copy link
Contributor

homu commented Nov 15, 2016

☔ The latest upstream changes (presumably #5176) made this pull request unmergeable. Please resolve the merge conflicts.

@chrismo
Copy link
Contributor

chrismo commented Nov 15, 2016

closing since this was taken care of in #5061

@chrismo chrismo closed this Nov 15, 2016
bundlerbot added a commit that referenced this pull request Nov 18, 2016
Improve and fixed guide style code from outdated cmd

Hi,

I separate these part from this PR #5058.

Regards.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants