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

yarn linc should also lint untracked files #11646

Closed
gaearon opened this issue Nov 23, 2017 · 5 comments
Closed

yarn linc should also lint untracked files #11646

gaearon opened this issue Nov 23, 2017 · 5 comments

Comments

@gaearon
Copy link
Collaborator

gaearon commented Nov 23, 2017

Too often I would run yarn linc, it would say lint passed for changed files, and then I'd realize I have not added a file yet so it doesn't see the file.

We should make yarn linc include untracked files. Note that if a file is ignored in git that's a different matter. But if it shows up as untracked it's extremely likely I want to lint it.

@accordeiro
Copy link
Contributor

Hi! Can I help with this one? :)

@gaearon
Copy link
Collaborator Author

gaearon commented Nov 23, 2017

Sure!

@accordeiro
Copy link
Contributor

Cool – I'll work on this and send a PR until this weekend.

@accordeiro
Copy link
Contributor

Just sent a PR – please let me know if this is what you were looking for :)

@gaearon
Copy link
Collaborator Author

gaearon commented Nov 27, 2017

Fixed in #11665.

@gaearon gaearon closed this as completed Nov 27, 2017
gaearon pushed a commit that referenced this issue Nov 27, 2017
* Lint untracked files with yarn linc (#11646)

* Run prettier on untracked files

* Unify code for listing changed files into shared utility
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants