Stuck on an issue?

Lightrun Answers was designed to reduce the constant googling that comes with debugging 3rd party libraries. It collects links to all the places you might be looking at while hunting down a tough bug.

And, if you’re still stuck at the end, we’re happy to hop on a call to see how we can help out.

Auto-fix by default?

See original GitHub issue

Most user’s workflow; run npm test, encounter linter errors, then manually run xo --fix.

Is there any good reason not to just make XO default to --fix? And instead have a xo --no-fix flag to opt-out. Are there any downsides in doing this?

This could be problematic for a few rules, like ava/no-skip-test, as its auto-fixer would remove the skip before running the test, but we could have a list of things to not auto-fix by default.

Issue Analytics

State:
Created 5 years ago
Reactions:5
Comments:11 (7 by maintainers)

Top GitHub Comments

1reaction

coreyfarrellcommented, Mar 22, 2019

If I make changes to index.js and run xo without running git add index.js then xo could end up modifying my unstaged file. If this were done I would add --no-fix to all my npm test targets.

1reaction

SamVerschuerencommented, Jan 25, 2019

Not sure about this. On one side, I like the idea because like you say, it can only help you. On the other hand, I think there are quite a lot of edge cases, some already mentioned by others where it works against you instead of with you.

A way to test this would be to publish some kind of pre-release version with --fix on by default and start using it.

Top Results From Across the Web

Auto Fix is enabled by default. Use the single string form

When I configure my vscode with eslint & prettier, I met a problem in .settings.json file with error message "Auto Fix is enabled...

How to change Auto Fix option for scheduled reports.

4) On the left hand side of the screen, locate Auto Fix under parameters. Right click and select edit. AutofixAttribute.PNG. 5) Select Default ......

AutoFix User's Manual

The default settings. Click on Start AutoFixing on the Settings panel to start automatic debugging, which involves first testing MY_LIST for 10 minutes...

I got this warning, Auto Fix is enabled by default. Use the ...

I got this warning, Auto Fix is enabled by default. Use the single string form. So, I changed it to,. "eslint.validate": [ "vue",...

Autofix Resources - Crafty

#What is this? In this guide we provide some simple examples for the usage of the auto-fix functionality. Crafty comes with some default...