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

Update README.md #878

Closed
wants to merge 1 commit into from
Closed

Update README.md #878

wants to merge 1 commit into from

Conversation

NickBusey
Copy link

Current command example fails with error Error: required flag(s) "pkg-name" not set

This fixes that, as well as fixing the package name to be consistent with the name lower down in the instructions.

@CLAassistant
Copy link

CLAassistant commented Jun 10, 2019

CLA assistant check
All committers have signed the CLA.

@Ghostwritten
Copy link

I thought it was my fault

@Ghostwritten
Copy link

Current command example fails with error Error: required flag(s) "pkg-name" not set

This fixes that, as well as fixing the package name to be consistent with the name lower down in the instructions.

how long?

@jharshman
Copy link
Collaborator

jharshman commented Jun 25, 2019

Bit of context: #817.
For completeness, you should include the following notes:

For adding commands, you should be inside your project root.
Init will no longer fail on non-empty directories.

@umarcor umarcor mentioned this pull request Jul 6, 2019
@jharshman jharshman mentioned this pull request Jul 10, 2019
@jharshman
Copy link
Collaborator

covered by #904

@jharshman jharshman closed this Jul 24, 2019
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.

4 participants