Skip to content

Conversation

@vnorekyan
Copy link

Addressing issue #58

@timfisher
Copy link

Is it possible to merge this please?

@Brynman
Copy link

Brynman commented Oct 7, 2020

We would also be very interested in this.

@vnorekyan
Copy link
Author

vnorekyan commented Oct 7, 2020

@timfisher @Brynman I don't have write access, so I can't merge this, can someone else with write access do that?

@vnorekyan vnorekyan changed the title Upgraded axe-core to 4.0.1 Upgraded axe-core to 4.0.2 Oct 7, 2020
Copy link
Member

@sapegin sapegin left a comment

Choose a reason for hiding this comment

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

Thanks for the pull request!

I've changed the way the code is bundled in #69, so axe-core isn't bundled anymore, but a peer dependency, it also allows axe-core 4.x.

@sapegin sapegin mentioned this pull request Oct 26, 2020
sapegin added a commit that referenced this pull request Oct 27, 2020
This allows consumers to use their own version of axe-core and significantly reduces the installation size. Also allow any 3.x or 4.x versions of axe-core in peer dependencies.

Closes #65, closes #59
@sapegin sapegin closed this in #69 Oct 27, 2020
sapegin added a commit that referenced this pull request Oct 27, 2020
This allows consumers to use their own version of axe-core and significantly reduces the installation size. Also allow any 3.x or 4.x versions of axe-core in peer dependencies.

Closes #65, closes #59
@github-actions
Copy link

🎉 This issue has been resolved in version 0.9.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