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

refact(cmd): version cmd from old cmd to new cmd #5655

Closed
wants to merge 1 commit into from
Closed

refact(cmd): version cmd from old cmd to new cmd #5655

wants to merge 1 commit into from

Conversation

qiwaa
Copy link
Contributor

@qiwaa qiwaa commented Oct 26, 2018

License: MIT
Signed-off-by: chenminjian 727180553@qq.com

@qiwaa qiwaa requested a review from Kubuxu as a code owner October 26, 2018 12:14
License: MIT
Signed-off-by: chenminjian <727180553@qq.com>
Copy link
Contributor

@schomatis schomatis left a comment

Choose a reason for hiding this comment

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

LGTM but I feel the --all option should supersede the rest (although this is not strictly related to this PR), if I do ipfs version --all --repo I just see the 7.

@qiwaa
Copy link
Contributor Author

qiwaa commented Oct 26, 2018

Good point! After this pr is merged, I will post an new issue about it. 😊

@Stebalien
Copy link
Member

I ended up merging #5650 (sorry, I should have checked for duplicates when I first asked @schomatis to review this).

I've created a meta-issue here to avoid this: #5664

@Stebalien Stebalien closed this Oct 26, 2018
@qiwaa
Copy link
Contributor Author

qiwaa commented Oct 27, 2018

Sorry, I didn't notice that someone had submitted this code before submitting pr.

@qiwaa qiwaa deleted the refact/cmd/version branch October 27, 2018 03:11
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.

3 participants