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

Add exit code information to cli docs #6049

Merged
merged 3 commits into from Apr 16, 2019

Conversation

jamesgeorge007
Copy link
Contributor

@jamesgeorge007 jamesgeorge007 commented Apr 11, 2019

Closes #5896

Add CLI existing status information the the docs.

  • I’ve added tests to confirm my change works.
  • (If changing the API or CLI) I’ve documented the changes I’ve made (in the docs/ directory)
  • (If the change is user-facing) I’ve added my changes to the CHANGELOG.unreleased.md file following the template.
  • I’ve read the contributing guidelines.

Add information regarding exit codes to the cli docs
@jamesgeorge007
Copy link
Contributor Author

@j-f1 Thoughts 🤔

@alexander-akait
Copy link
Member

alexander-akait commented Apr 11, 2019

Please fix linting problems

 prettylint {.,docs,website,website/blog}/*.md

  docs/cli.md:57:21
  ⚠  57:21  Insert ························                                                            prettier/prettier
  ⚠  58:9   Insert ------------------------                                                            prettier/prettier
  ⚠  59:3   Replace ·|·Everything·formatted·properly· with ····|·Everything·formatted·properly·······  prettier/prettier
  ⚠  60:4   Insert ···                                                                                 prettier/prettier
  ⚠  61:4   Replace |·Something's·wrong·with·Prettier with ···|·Something's·wrong·with·Prettier····    prettier/prettier

  5 warnings

@j-f1 j-f1 merged commit ed15b6d into prettier:master Apr 16, 2019
@j-f1
Copy link
Member

j-f1 commented Apr 16, 2019

Thanks for contributing to Prettier!

@jamesgeorge007 jamesgeorge007 deleted the hotfix/update-docs branch April 16, 2019 12:09
@lock lock bot added the locked-due-to-inactivity Please open a new issue and fill out the template instead of commenting. label Jul 15, 2019
@lock lock bot locked as resolved and limited conversation to collaborators Jul 15, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
locked-due-to-inactivity Please open a new issue and fill out the template instead of commenting.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

CLI exit status not documented
3 participants