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

Use suggestions API for no-only-test and no-skip-test #325

Merged
merged 2 commits into from
Mar 24, 2021

Conversation

edbrannin
Copy link
Contributor

@edbrannin edbrannin commented Mar 19, 2021

@edbrannin
Copy link
Contributor Author

@sindresorhus Integration tests are failing in the same way as they did in #324. Was the build already broken?

@sindresorhus
Copy link
Member

You need to update the rule docs and the readme.

@edbrannin
Copy link
Contributor Author

Whoops, thanks! I'll fix.

...actually, I checked the eslint code rules, the only two that support the suggest: API don't say anything about that or autofix in the docs.

Should I just drop references to Fixable for these rules, or would you prefer some other language?

@sindresorhus
Copy link
Member

Should I just drop references to Fixable for these rules, or would you prefer some other language?

Drop

@edbrannin
Copy link
Contributor Author

Done

(oh yay, the integration test is working now too!)

@sindresorhus sindresorhus changed the title Use suggestions API for no-only-test, no-skip-test Use suggestions API for no-only-test and no-skip-test Mar 24, 2021
@sindresorhus sindresorhus merged commit ae34aea into avajs:main Mar 24, 2021
@edbrannin
Copy link
Contributor Author

Thanks!

If you don't mind my asking, any idea when this might be released? I'm very much looking forward to updating this plugin on my services.

@sindresorhus
Copy link
Member

It was released half an hour ago.

@edbrannin
Copy link
Contributor Author

Oh, double-thanks then! Keep up the good work!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants