Skip to content

Remove pa11y from CI / Makefile#799

Merged
jessieay merged 1 commit intomasterfrom
jy-remove-pa11y-crawl
Dec 1, 2016
Merged

Remove pa11y from CI / Makefile#799
jessieay merged 1 commit intomasterfrom
jy-remove-pa11y-crawl

Conversation

@jessieay
Copy link
Contributor

@jessieay jessieay commented Dec 1, 2016

Why:

  • We are now testing accessibility of all pages through axe-matchers /
    RSpec so Pa11y is duplicative

package.json Outdated
Copy link
Contributor

Choose a reason for hiding this comment

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

we just want to remove the scripts, right? Not remove the entire package.json?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

hah ooops I think I may have been to aggressive here.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

re-added the stuff we still use and force pushed :)

@jessieay jessieay force-pushed the jy-remove-pa11y-crawl branch from a0a2a82 to 05da1e3 Compare December 1, 2016 19:34
Copy link
Contributor

@zachmargolis zachmargolis left a comment

Choose a reason for hiding this comment

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

LGTM!

@jessieay jessieay force-pushed the jy-remove-pa11y-crawl branch from 05da1e3 to f4a843d Compare December 1, 2016 19:37
**Why**:
* We are now testing accessibility of all pages through axe-matchers /
  RSpec so Pa11y is duplicative
@jessieay jessieay self-assigned this Dec 1, 2016
@jessieay jessieay merged commit 3f0c14c into master Dec 1, 2016
@jessieay jessieay deleted the jy-remove-pa11y-crawl branch December 1, 2016 19:45
amoose pushed a commit that referenced this pull request Feb 24, 2017
**Why**:
* We are now testing accessibility of all pages through axe-matchers /
  RSpec so Pa11y is duplicative
amoose pushed a commit that referenced this pull request Feb 28, 2017
**Why**:
* We are now testing accessibility of all pages through axe-matchers /
  RSpec so Pa11y is duplicative
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants