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

feat(axe-core-4.9.1): update to use axe-core 4.9.1 #1322

Merged
merged 6 commits into from
Jun 20, 2024

Conversation

rohittarpara
Copy link
Contributor

@rohittarpara rohittarpara commented Jun 18, 2024

Details

This PR upgrades axe-core to 4.9.1 according to the instructions in CONTRIBUTING.md.

Motivation

feature work 🚀

Context

Pull request checklist

  • PR title respects Conventional Commits (starts with fix:, feat:, etc, and is suitable for user-facing release notes)
  • PR contains no breaking changes, OR description of both PR and final merge commit starts with BREAKING CHANGE:
  • Verify PR title and final merge commit contain space after : for example feat(feature): feature title otherwise it will not be considered by semantic-release for release.
  • (if applicable) Addresses issue: #0000
  • Added relevant unit tests for your changes
  • Ran yarn precheckin
  • Verified code coverage for the changes made

@rohittarpara rohittarpara requested a review from a team as a code owner June 18, 2024 11:37
Copy link
Contributor

@madalynrose madalynrose left a comment

Choose a reason for hiding this comment

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

Per the instructions in CONTRIBUTING.md, I think we also need to update the devDependencies in package.json for @axe-core/cli and @axe-core/puppeteer. And then regenerate the resources. Additionally, src/cli.test.ts needs to be updated similar to how src/index.test.ts was updated.

@rohittarpara
Copy link
Contributor Author

rohittarpara commented Jun 19, 2024

@madalynrose i updated version for @axe-core/cli and @axe-core/puppeteer.

@v-viyada v-viyada changed the title feat(axe-core-4.9.1):update to use axe-core 4.9.1 feat(axe-core-4.9.1): update to use axe-core 4.9.1 Jun 19, 2024
Copy link
Contributor

@madalynrose madalynrose left a comment

Choose a reason for hiding this comment

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

one small adjustment but this looks good!

package.json Outdated Show resolved Hide resolved
@v-viyada
Copy link
Contributor

/azp run

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@v-viyada v-viyada merged commit 24ef1fe into microsoft:main Jun 20, 2024
4 checks passed
@ada-cat
Copy link
Collaborator

ada-cat commented Jun 27, 2024

🎉 This PR is included in version 2.12.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

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

Successfully merging this pull request may close these issues.

4 participants