Skip to content
This repository has been archived by the owner on Feb 12, 2022. It is now read-only.

Documentation fixes #194

Open
wants to merge 7 commits into
base: master
Choose a base branch
from
Open

Documentation fixes #194

wants to merge 7 commits into from

Conversation

KarateCowboy
Copy link
Contributor

No description provided.

@codecov
Copy link

codecov bot commented Sep 7, 2017

Codecov Report

Merging #194 into master will increase coverage by 0.05%.
The diff coverage is 100%.

Impacted file tree graph

@@            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
Impacted Files Coverage Δ
src/commands/apps/errors.js 6.89% <ø> (ø) ⬆️
src/commands/auth/login.js 20% <ø> (ø) ⬆️
src/commands/features/disable.js 30% <ø> (ø) ⬆️
src/commands/auth/whoami.js 90% <ø> (ø) ⬆️
src/commands/buildpacks/add.js 28.57% <ø> (ø) ⬆️
src/commands/ps/type.js 15% <ø> (ø) ⬆️
src/commands/maintenance/on.js 42.85% <ø> (ø) ⬆️
src/commands/config/index.js 20% <ø> (ø) ⬆️
src/commands/labs/index.js 10.34% <ø> (ø) ⬆️
src/commands/labs/disable.js 16.66% <ø> (ø) ⬆️
... and 35 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e57b303...1a70b78. Read the comment docs.

Example:

$ heroku apps:destroy --app murmuring-headland-14719
▸ WARNING: This will delete ⬢ murmuring-headland-14719 including all add-ons.
Copy link
Contributor

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

example
example2
$ === My Apps
$ example
Copy link
Contributor

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?


$ heroku 2fa:generate-recovery-codes
$ Password: ***************
$ Recovery codes:
Copy link
Contributor

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


$ heroku 2fa
$ Two-factor authentication is enabled`
Copy link
Contributor

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`
Copy link
Contributor

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

Copy link
Contributor Author

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.

run: cli.command(co.wrap(run)),
help: `Example:

heroku labs:disable app-alerting --app murmuring-headland-14719
Copy link
Contributor

Choose a reason for hiding this comment

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

dollar sign

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants