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

Pin eslint-plugin-jsx-a11y to ^6.7.1 everywhere it's referenced #5794

Merged
merged 4 commits into from
May 7, 2024

Conversation

ichim-david
Copy link
Member

@ichim-david ichim-david commented Feb 24, 2024

Fixes #5785

  • Also added the disabling "jsx-a11y/label-has-associated-control" in case it reaches version 6.8.0 as we have failures within Volto

- Also added the disabling "jsx-a11y/label-has-associated-control" in case it reaches version 6.8.0 as we have failures within Volto
Copy link

netlify bot commented Feb 24, 2024

Deploy Preview for volto canceled.

Name Link
🔨 Latest commit 027b05e
🔍 Latest deploy log https://app.netlify.com/sites/volto/deploys/6638ee22cbf0860008170b04

Copy link

netlify bot commented Feb 24, 2024

Deploy Preview for plone-components canceled.

Name Link
🔨 Latest commit 027b05e
🔍 Latest deploy log https://app.netlify.com/sites/plone-components/deploys/6638ee22884dfb0009e032b2

@@ -27,7 +27,8 @@
],
"react-hooks/rules-of-hooks": "error",
"react-hooks/exhaustive-deps": "warn",
"react/react-in-jsx-scope": "off"
"react/react-in-jsx-scope": "off",
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

For 18 alpha is not really required, we only need to add an entry to the upgrade guide.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

(and fix the violations)

@sneridagh sneridagh merged commit 9e4f05f into main May 7, 2024
73 checks passed
@sneridagh sneridagh deleted the 5785-jxs-a11y branch May 7, 2024 16:24
sneridagh added a commit that referenced this pull request May 13, 2024
* main:
  Revert "Fix 301 and 302 redirects. (#5999)" (#6008)
  Pin eslint-plugin-jsx-a11y to ^6.7.1 everywhere it's referenced (#5794)
  Allow Makefile to be modified by Makefile.local if present (#5997)
  Cypress replaced wait with assertions (#5998)
  Replace spinner with Progress bar and fix it's position . (#5620) (#5929)
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.

eslint-plugin-jsx-a11y pin from Volto shouldn't have exact version pins
2 participants