-
-
Notifications
You must be signed in to change notification settings - Fork 32.4k
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
Bump eslint-plugin-jsx-a11y from 6.2.3 to 6.3.1 #21525
Bump eslint-plugin-jsx-a11y from 6.2.3 to 6.3.1 #21525
Conversation
@dependabot rebase |
Looks like this PR has been edited by someone other than Dependabot. That means Dependabot can't rebase it - sorry! If you're happy for Dependabot to recreate it from scratch, overwriting any edits, you can request |
@dependabot recreate |
Bumps [eslint-plugin-jsx-a11y](https://github.com/evcohen/eslint-plugin-jsx-a11y) from 6.2.3 to 6.3.1. - [Release notes](https://github.com/evcohen/eslint-plugin-jsx-a11y/releases) - [Changelog](https://github.com/jsx-eslint/eslint-plugin-jsx-a11y/blob/master/CHANGELOG.md) - [Commits](infofarmer/eslint-plugin-jsx-a11y@v6.2.3...v6.3.1) Signed-off-by: dependabot-preview[bot] <support@dependabot.com>
453b49f
to
4ea6af8
Compare
@@ -137,8 +137,8 @@ export default function ConfirmationDialog() { | |||
|
|||
return ( | |||
<div className={classes.root}> | |||
<List component="div" role="list"> | |||
<ListItem button divider disabled role="listitem"> | |||
<List component="div" role="group"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Makes more sense to group them with the current implementation of List (related to #13597 (comment)). I'd question the demo as a whole but this requires more context. It's using List for styles and has to go the extra mile to remove the implied semantics.
Let's leave it as is to get the build green and revisit this when working on #13597
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@dependabot squash and merge
Bumps eslint-plugin-jsx-a11y from 6.2.3 to 6.3.1.
Changelog
Sourced from eslint-plugin-jsx-a11y's changelog.
Commits
1614b85
CHANGELOG v6.3.1b82b1bd
Merge pull request #705 from jsx-eslint/fix-heading-error765da0f
Update to aria-query 4.2.2d528e8c
Fix aria-level allowed on elements wit role heading (#704)3ac1738
Merge pull request #674 from jsx-eslint/greenkeeper/estraverse-5.0.0bbfcb5a
Merge branch 'master' into greenkeeper/estraverse-5.0.029c6859
[meta] remove yarn registry from npmrc, so publishing works7b5d228
Update CHANGELOG for v6.3.0e215b87
Merge pull request #696 from jsx-eslint/fix-aria-1-2-test-failurescce838a
Update aria-query to 4.2.0Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting
@dependabot rebase
.Dependabot commands and options
You can trigger Dependabot actions by commenting on this PR:
@dependabot rebase
will rebase this PR@dependabot recreate
will recreate this PR, overwriting any edits that have been made to it@dependabot merge
will merge this PR after your CI passes on it@dependabot squash and merge
will squash and merge this PR after your CI passes on it@dependabot cancel merge
will cancel a previously requested merge and block automerging@dependabot reopen
will reopen this PR if it is closed@dependabot close
will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually@dependabot badge me
will comment on this PR with code to add a "Dependabot enabled" badge to your readmeAdditionally, you can set the following in the
.dependabot/config.yml
file in this repo: