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 tap-snapshots/**
to default ignores
#384
Comments
Sure. My general rule for adding default ignores is that it has to be helpful for a lot of users and that it's unambiguous. |
coreyfarrell
added a commit
to coreyfarrell/xo
that referenced
this issue
May 21, 2019
sindresorhus
pushed a commit
that referenced
this issue
May 29, 2019
coreyfarrell
added a commit
to coreyfarrell/xo
that referenced
this issue
Apr 10, 2020
The next version of tap will be using the `.cjs` extension to for compatibility with packages that have `"type": "module"`. Ref xojs#384
coreyfarrell
added a commit
to coreyfarrell/xo
that referenced
this issue
Apr 10, 2020
The next version of tap will be using the `.cjs` extension for compatibility with packages that have `"type": "module"`. Ref xojs#384
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Would you accept a patch adding
tap-snapshots/*.js
to the default ignores? These files are generated/used by tapjs. I set thexo.ignores
option in package.json in a project using tap snapshots but hope this scenario can be supported with zero config.The text was updated successfully, but these errors were encountered: