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

Don't auto-commit changed files #41

Closed
ajwagnerloomis opened this issue Sep 21, 2018 · 2 comments
Closed

Don't auto-commit changed files #41

ajwagnerloomis opened this issue Sep 21, 2018 · 2 comments

Comments

@ajwagnerloomis
Copy link

Is it possible to tell pretty-quick to not commit files that have been changed automatically? I would prefer to be able to review the changes before committing them.

I tried the --no-restage option as implemented with #39, but that ended up committing the file any way. Am I misunderstanding that flag?

I am happy to do this as a PR, but wanted to make sure that I understood the problem and make sure it wasn't already fixed.

@azz
Copy link
Member

azz commented Sep 22, 2018

pretty-quick might not be the right tool for that kind of workflow. It might be best to run prettier directly.

@azz azz closed this as completed Oct 20, 2018
@jorgeorpinel
Copy link

--bail does this I think. But it's confusing as --no-restage sounds like it will also have the same effect...

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

No branches or pull requests

3 participants