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

Remove Playwright-Axe testing code #104

Merged
merged 3 commits into from
Jul 21, 2022
Merged

Remove Playwright-Axe testing code #104

merged 3 commits into from
Jul 21, 2022

Conversation

gabalafou
Copy link
Collaborator

@gabalafou gabalafou commented Jul 21, 2022

This PR in conjunction with PR Quansight-Labs/jupyter-a11y-testing#1 closes issue Quansight-Labs/jupyter-a11y-mgmt#144.

The goal of this pull request is to remove all of the automated testing code that @tonyfast, @trallard and I (@gabalafou) added in over the past few months.

We are moving that work into its own repo, as part of the larger task of restructuring this repo.

This pull request, which removes the code from this repo, should be the opposite Quansight-Labs/jupyter-a11y-testing#1, which adds the code.

This pull request is essentially the result of undoing two earlier pull requests:

Revert "Merge pull request #92 from Quansight-Labs/master"

This reverts commit ed2254d, reversing
changes made to 87e4467.
This reverts commit 0556994, reversing
changes made to ed2254d.
@gabalafou gabalafou changed the title Move testing code Remove Playwright-Axe testing code Jul 21, 2022
@gabalafou gabalafou marked this pull request as ready for review July 21, 2022 15:42
@gabalafou gabalafou requested a review from trallard July 21, 2022 15:42
Copy link
Member

@trallard trallard left a comment

Choose a reason for hiding this comment

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

Ok let's merge and hope for the best 🤞🏽

@trallard trallard merged commit e5656a6 into main Jul 21, 2022
@trallard trallard deleted the move-testing-code branch July 21, 2022 17:37
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.

2 participants