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

Detect config file mishaps #199

Open
novemberborn opened this issue Jul 14, 2018 · 4 comments
Open

Detect config file mishaps #199

novemberborn opened this issue Jul 14, 2018 · 4 comments
Labels
💵 Funded on Issuehunt This issue has been funded on Issuehunt enhancement help wanted

Comments

@novemberborn
Copy link
Member

novemberborn commented Jul 14, 2018

Issuehunt badges

#192 adds support for ava.config.js files. There are possible bad configurations where AVA itself will refuse to run tests, that are ignored by this ESLint plugin.

It'd be good (if possible?) to have a lint rule that checks for those problems so we can issue a warning (or error).


IssueHunt Summary

Sponsors (Total: $60.00)

Become a sponsor now!

Or submit a pull request to get the deposits!

Tips

@sindresorhus
Copy link
Member

What kind of bad configurations? Invalid options?

@novemberborn
Copy link
Member Author

Config in package.json#ava with an ava.config.js present, or the latter file exporting bad values.

@issuehunt-oss issuehunt-oss bot added the 💵 Funded on Issuehunt This issue has been funded on Issuehunt label Jun 9, 2019
@IssueHuntBot
Copy link

@issuehunt has funded $60.00 to this issue.


@novemberborn
Copy link
Member Author

We should add a standardized way of communicating these errors to https://github.com/avajs/ava/blob/master/eslint-plugin-helper.js, and then surface them in this plugin.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
💵 Funded on Issuehunt This issue has been funded on Issuehunt enhancement help wanted
Projects
None yet
Development

No branches or pull requests

3 participants