-
Notifications
You must be signed in to change notification settings - Fork 2.9k
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
Small correction in README #1009
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think that the second commit should be a separate PR, but both of them are ok.
@nickolashkraus, thanks! That allowed to rebase many other PRs that failed because of the year. |
@umarcor Anything else required from me on this PR? |
This PR is being marked as stale due to a long period of inactivity |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @nickolashkraus - thank you so much for the contribution!!
Looks like there is a conflict in the readme. Can you update and I'll get this in ASAP?
Hi @nickolashkraus - any chance we can get an update from you on this? Otherwise, I'll close this and open an issue to address this - thanks much |
@jpmcb Sorry I missed your original message! Rebased and addressed merge conflicts |
I thought this sentence seemed off:
In the Viper documentation:
One would assume that:
Testing with
cobra init
bears this out:cobra init --pkg-name test
cobra init --pkg-name test --author Me