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

deprecate the bundle viz command #5964

Merged
merged 1 commit into from Sep 6, 2017
Merged

Conversation

colby-swandale
Copy link
Member

@colby-swandale colby-swandale commented Aug 22, 2017

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

The bundle viz command is to been removed from bundler 2 and extracted into a plugin

Closes #5180

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

Deprecate the bundle viz command with an error explaining that a new gem will replace it's functionality.

@segiddins
Copy link
Member

segiddins commented Aug 22, 2017

We also need a feature flag around the definition of the command in CLI so that a plugin can pick it up (or just call self.class.handle_no_command_error("viz") and try to make the UX nice for both when the plugin is installed and when it isn't)

This also requires a bundle-vis gem to be published

@indirect indirect added this to the 2.0 — Breaking Changes milestone Aug 29, 2017
@segiddins
Copy link
Member

Cannot use viz_command as a settings feature flag since it isn't a bool key

@bundlerbot
Copy link
Collaborator

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

@segiddins
Copy link
Member

@indirect thoughts?

@indirect
Copy link
Member

indirect commented Sep 5, 2017

@segiddins This works for me. If @colby-swandale is too busy to finish it, can you rebase it and fix the bool key thing?

@segiddins
Copy link
Member

Bye-bye bundle viz

@bundlerbot r+

@bundlerbot
Copy link
Collaborator

📌 Commit 0b04562 has been approved by segiddins

@bundlerbot
Copy link
Collaborator

⌛ Testing commit 0b04562 with merge dbb0aed...

bundlerbot added a commit that referenced this pull request Sep 6, 2017
deprecate the bundle viz command

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

The `bundle viz` command is to been removed from bundler 2 and extracted into a plugin

Closes #5180

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

Deprecate the `bundle viz` command with an error explaining that a new gem will replace it's functionality.
@bundlerbot
Copy link
Collaborator

☀️ Test successful - status-travis
Approved by: segiddins
Pushing dbb0aed to master...

@bundlerbot bundlerbot merged commit 0b04562 into master Sep 6, 2017
@colby-swandale colby-swandale deleted the colby/deprecate-viz-command branch September 6, 2017 20:53
colby-swandale pushed a commit that referenced this pull request Sep 22, 2018
deprecate the bundle viz command

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

The `bundle viz` command is to been removed from bundler 2 and extracted into a plugin

Closes #5180

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

Deprecate the `bundle viz` command with an error explaining that a new gem will replace it's functionality.

(cherry picked from commit dbb0aed)
colby-swandale pushed a commit that referenced this pull request Sep 22, 2018
deprecate the bundle viz command

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

The `bundle viz` command is to been removed from bundler 2 and extracted into a plugin

Closes #5180

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

Deprecate the `bundle viz` command with an error explaining that a new gem will replace it's functionality.

(cherry picked from commit dbb0aed)
colby-swandale pushed a commit that referenced this pull request Sep 22, 2018
deprecate the bundle viz command

The `bundle viz` command is to been removed from bundler 2 and extracted into a plugin

Closes #5180

Deprecate the `bundle viz` command with an error explaining that a new gem will replace it's functionality.

(cherry picked from commit dbb0aed)
colby-swandale pushed a commit that referenced this pull request Oct 5, 2018
deprecate the bundle viz command

The `bundle viz` command is to been removed from bundler 2 and extracted into a plugin

Closes #5180

Deprecate the `bundle viz` command with an error explaining that a new gem will replace it's functionality.

(cherry picked from commit dbb0aed)
colby-swandale added 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.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Move bundle viz into a plugin
5 participants