Skip to content

feat: deprecate no-onchange rule #854

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Merged
merged 1 commit into from
Jul 24, 2023
Merged

feat: deprecate no-onchange rule #854

merged 1 commit into from
Jul 24, 2023

Conversation

Lehoczky
Copy link
Contributor

🔗 Linked Issue

Resolves #97

Description

What has been done:

  • the rule has been removed from the recommended config
  • the rule has been marked as deprecated in the rule config itself
  • the documentation has been updated to describe why the rule has been deprecated. I used the same description as the eslint-plugin-jsx-a11y and open-wc docs are using

@vhoyer vhoyer merged commit 3fe0f5b into vue-a11y:main Jul 24, 2023
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Do we need a no-onchange rule?
2 participants