8000 Add plugin list command by jumbosushi · Pull Request #6120 · rubygems/bundler · GitHub
[go: up one dir, main page]
Skip to content
This repository was archived by the owner on Apr 14, 2021. It is now read-only.

Add plugin list command #6120

Merged
bundlerbot merged 2 commits into from
Oct 22, 2017
Merged

Add plugin list command #6120

bundlerbot merged 2 commits into from
Oct 22, 2017

Conversation

@jumbosushi
Copy link
Contributor
@jumbosushi jumbosushi commented Oct 22, 2017 •
edited
Loading

This is a continuation of PR #5672 which was originally created to fix Issue #5467

Fixes the last PR, and add additional test files to check Bundler::Plugin#list

@ghost
Copy link
ghost commented Oct 22, 2017

Thanks for opening a pull request and helping make Bundler better! Someone from the Bundler team will take a look at your pull request shortly and leave any feedback. Please make sure that your pull request has tests for any changes or added functionality.

We use Travis CI to test and make sure your change works functionally and uses acceptable conventions, you can review the current progress of Travis CI in the PR status window below.

If you have any questions or concerns that you wish to ask, feel free to leave a comment in this PR or join our #bundler channel on Slack.

For more information about contributing to the Bundler project feel free to review our CONTRIBUTING guide

@jumbosushi jumbosushi changed the title Plugin list Add plugin list command Oct 22, 2017
@indirect
Copy link
Contributor
indirect commented Oct 22, 2017

This looks great, thanks so much for picking this up and finishing it.

@bundlerbot r+

@bundlerbot
Copy link
Collaborator
bundlerbot commented Oct 22, 2017

📌 Commit e1dfa07 has been approved by indirect

@bundlerbot
Copy link
Collaborator
bundlerbot commented Oct 22, 2017

⌛ Testing commit e1dfa07 with merge 8aea214...

bundlerbot added a commit that referenced this pull request Oct 22, 2017
@bundlerbot
Auto merge of #6120 - jumbosushi:plugin_list, r=indirect
8aea214 
Add plugin list command

This is a continuation of PR #5672  which was originally created to fix Issue #5467

Fixes a typo from the last PR, and add additional test files to check Bundler::Plugin#list
@bundlerbot
Copy link
Collaborator
bundlerbot commented Oct 22, 2017

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

@bundlerbot bundlerbot merged commit e1dfa07 into rubygems:master Oct 22, 2017
@bundlerbot bundlerbot mentioned this pull request Oct 22, 2017
Closed
@colby-swandale colby-swandale mentioned this pull request Oct 23, 2017
Merged
@jumbosushi
Copy link
Contributor Author
jumbosushi commented Oct 24, 2017

Thanks for the merge! @indirect Can you update #4871 checklist?

@andrew andrew mentioned this pull request Feb 19, 2019
Closed
@jules2689 jules2689 mentioned this pull request Mar 14, 2020
Open
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@jumbosushi @indirect @bundlerbot @fotanus
0