-
Notifications
You must be signed in to change notification settings - Fork 26
Documentation fixes #194
base: master
Are you sure you want to change the base?
Documentation fixes #194
Conversation
Codecov Report
@@ Coverage Diff @@
## master #194 +/- ##
==========================================
+ Coverage 19.32% 19.37% +0.05%
==========================================
Files 72 72
Lines 1449 1450 +1
Branches 276 276
==========================================
+ Hits 280 281 +1
Misses 913 913
Partials 256 256
Continue to review full report at Codecov.
|
src/commands/apps/destroy.js
Outdated
Example: | ||
|
||
$ heroku apps:destroy --app murmuring-headland-14719 | ||
▸ WARNING: This will delete ⬢ murmuring-headland-14719 including all add-ons. |
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.
this should be indented further
src/commands/apps/index.js
Outdated
example | ||
example2 | ||
$ === My Apps | ||
$ example |
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.
should there be dollar signs here?
src/commands/auth/2fa/generate.js
Outdated
|
||
$ heroku 2fa:generate-recovery-codes | ||
$ Password: *************** | ||
$ Recovery codes: |
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.
shouldn't have dollar signs here
src/commands/auth/2fa/index.js
Outdated
|
||
$ heroku 2fa | ||
$ Two-factor authentication is enabled` |
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.
dollar signs
help: `Example: | ||
|
||
$ heroku auth:logout | ||
Local credentials cleared` |
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.
This output should be different since it's actually logging the user out, but this is an issue with the command not your documentation
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 see what you mean. That was the actual output I captured.
src/commands/labs/disable.js
Outdated
run: cli.command(co.wrap(run)), | ||
help: `Example: | ||
|
||
heroku labs:disable app-alerting --app murmuring-headland-14719 |
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.
dollar sign
No description provided.