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

[2.0] [Resolver] Use the GVP for major updates in 2.0#5995

Merged
bundlerbot merged 3 commits intomasterfrom
seg-gvp-major
Sep 5, 2017
Merged

[2.0] [Resolver] Use the GVP for major updates in 2.0#5995
bundlerbot merged 3 commits intomasterfrom
seg-gvp-major

Conversation

@segiddins
Copy link
Copy Markdown
Contributor

Thanks so much for the contribution!
To make reviewing this PR a bit easier, please fill out answers to the following questions.

What was the end-user problem that led to this PR?

The problem was @chrismo wants to start delegating all sorting to the GVP.

What was your diagnosis of the problem?

My diagnosis was to start doing that in 2.0.

Closes #5993

What is your fix for the problem, implemented in this PR?

My fix adds a feature flag.

@segiddins segiddins requested a review from chrismo August 31, 2017 23:00
@segiddins
Copy link
Copy Markdown
Contributor Author

This seems to break bundler/inline and I'm not quite sure why yet

@segiddins
Copy link
Copy Markdown
Contributor Author

This ought to pass now!

r?

@indirect
Copy link
Copy Markdown

indirect commented Sep 5, 2017

@bundlerbot r+

@bundlerbot
Copy link
Copy Markdown
Collaborator

📌 Commit 092dfb6 has been approved by indirect

@bundlerbot
Copy link
Copy Markdown
Collaborator

⌛ Testing commit 092dfb6 with merge a325b6d...

bundlerbot added a commit that referenced this pull request Sep 5, 2017
[2.0] [Resolver] Use the GVP for major updates in 2.0

Thanks so much for the contribution!
To make reviewing this PR a bit easier, please fill out answers to the following questions.

### What was the end-user problem that led to this PR?

The problem was @chrismo wants to start delegating all sorting to the GVP.

### What was your diagnosis of the problem?

My diagnosis was to start doing that in 2.0.

Closes #5993

### What is your fix for the problem, implemented in this PR?

My fix adds a feature flag.
@bundlerbot
Copy link
Copy Markdown
Collaborator

☀️ Test successful - status-travis
Approved by: indirect
Pushing a325b6d to master...

@bundlerbot bundlerbot merged commit 092dfb6 into master Sep 5, 2017
@segiddins segiddins deleted the seg-gvp-major branch September 5, 2017 20:38
@chrismo
Copy link
Copy Markdown
Contributor

chrismo commented Sep 7, 2017

thx a bunch @segiddins

@deivid-rodriguez deivid-rodriguez added this to the 1.17.0 milestone Sep 3, 2018
colby-swandale pushed a commit that referenced this pull request Sep 22, 2018
[2.0] [Resolver] Use the GVP for major updates in 2.0

Thanks so much for the contribution!
To make reviewing this PR a bit easier, please fill out answers to the following questions.

The problem was @chrismo wants to start delegating all sorting to the GVP.

My diagnosis was to start doing that in 2.0.

Closes #5993

My fix adds a feature flag.

(cherry picked from commit a325b6d)
colby-swandale pushed a commit that referenced this pull request Sep 22, 2018
[2.0] [Resolver] Use the GVP for major updates in 2.0

Thanks so much for the contribution!
To make reviewing this PR a bit easier, please fill out answers to the following questions.

The problem was @chrismo wants to start delegating all sorting to the GVP.

My diagnosis was to start doing that in 2.0.

Closes #5993

My fix adds a feature flag.

(cherry picked from commit a325b6d)
colby-swandale pushed a commit that referenced this pull request Oct 5, 2018
[2.0] [Resolver] Use the GVP for major updates in 2.0

Thanks so much for the contribution!
To make reviewing this PR a bit easier, please fill out answers to the following questions.

The problem was @chrismo wants to start delegating all sorting to the GVP.

My diagnosis was to start doing that in 2.0.

Closes #5993

My fix adds a feature flag.

(cherry picked from commit a325b6d)
colby-swandale pushed a commit that referenced this pull request Oct 25, 2018
* 1-17-stable: (38 commits)
  Version 1.17.0 with changelog
  Merge #6754
  Version 1.17.0.pre.2 with changelog
  fix failing bundle remove specs
  Still document the `--force` option
  scope specs testing bundler 2 deprecations to bundler 1 only
  Merge #6718
  Merge #6707
  Merge #6702
  Merge #6316
  Auto merge of #6447 - agrim123:agr-update-error-message, r=segiddins
  Auto merge of #6513 - agrim123:agr-bundler-remove, r=indirect
  Auto merge of #6318 - jhawthorn:fix_incorrect_test_in_requires_sudo, r=segiddins
  Auto merge of #6450 - bundler:segiddins/bundle-binstubs-all, r=colby-swandale
  Auto merge of #6024 - gwerbin:custom-user-bundle-dir, r=colby-swandale
  Version 1.17.0.pre.1 with changelog
  Auto merge of #5964 - bundler:colby/deprecate-viz-command, r=segiddins
  Auto merge of #5986 - bundler:seg-jobs-count, r=indirect
  Auto merge of #5995 - bundler:seg-gvp-major, r=indirect
  Auto merge of #5803 - igorbozato:path_relative_to_pwd, r=indirect
  ...
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.

Enable GemVersionPromoter for all cases

5 participants