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.

Sign up for GitHub

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

Jump to bottom

Switch from DRY_RUN to --fix #22

Open
tarkatronic opened this issue Feb 5, 2021 · 0 comments
Open

Switch from DRY_RUN to --fix #22

tarkatronic opened this issue Feb 5, 2021 · 0 comments
Labels
enhancement New feature or request fixers Related to one or more of the fixers help wanted Extra attention is needed

Comments

@tarkatronic
Copy link
Contributor

In keeping with the thought path of this being a linter we should, by default, only report out on errors. That somewhat obviates the mechanism of a "dry run", and instead changes us to the more explicit operation of a "fix", such as with eslint --fix.

This will probably have to rely on first implementing an actual CLI library such as oclif, yargs or commander. I don't know the pros/cons of each, so happy to take recommendations.
@tarkatronic tarkatronic added enhancement New feature or request help wanted Extra attention is needed fixers Related to one or more of the fixers labels Feb 5, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request fixers Related to one or more of the fixers help wanted Extra attention is needed
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@tarkatronic