Skip to content

Pin axe-core version #666

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

Merged
merged 1 commit into from
Feb 26, 2020
Merged

Conversation

matijs
Copy link
Contributor

@matijs matijs commented Feb 24, 2020

Pin axe-core to v3.4.1 which is the latest version in which tests for
the 'autocomplete-valid' rule pass.

See this PR for more
information and small discussion.

Pin axe-core to v3.4.1 which is the latest version in which tests for the 'autocomplete-valid' rule pass.

See jsx-eslint#665 and dequelabs/axe-core#2060 for more information and small discussion.
Copy link
Member

@ljharb ljharb left a comment

Choose a reason for hiding this comment

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

Has an issue been filed upstream on axe-core for its breaking change in a patch?

@coveralls
Copy link

coveralls commented Feb 24, 2020

Coverage Status

Coverage remained the same at 99.484% when pulling f13dc38 on matijs:fix/pin-axe-core-version into 46369c3 on evcohen:master.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 99.484% when pulling 7f3844f on matijs:fix/pin-axe-core-version into 46369c3 on evcohen:master.

@matijs matijs force-pushed the fix/pin-axe-core-version branch from 7f3844f to 43dbd29 Compare February 24, 2020 10:11
@matijs
Copy link
Contributor Author

matijs commented Feb 24, 2020

Has an issue been filed upstream on axe-core for its breaking change in a patch?

I haven't had time yet.

@ljharb
Copy link
Member

ljharb commented Feb 24, 2020

Let’s do that before merging here, so we don’t forget.

@matijs
Copy link
Contributor Author

matijs commented Feb 25, 2020

I filed an issue

@ljharb ljharb force-pushed the fix/pin-axe-core-version branch from 43dbd29 to f13dc38 Compare February 26, 2020 00:20
@ljharb ljharb merged commit f13dc38 into jsx-eslint:master Feb 26, 2020
@ljharb
Copy link
Member

ljharb commented Feb 26, 2020

thanks for this "beast" of a PR :-p

@matijs matijs deleted the fix/pin-axe-core-version branch February 26, 2020 09:11
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.

3 participants