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

🧹 Consolidate/finalise linting; consider axe Linting and Linkinator #202

Closed
4 tasks
tatianamac opened this issue Jun 10, 2020 · 9 comments · Fixed by #253
Closed
4 tasks

🧹 Consolidate/finalise linting; consider axe Linting and Linkinator #202

tatianamac opened this issue Jun 10, 2020 · 9 comments · Fixed by #253
Assignees
Labels
Status · In Progress Type · DX Affects developer experience

Comments

@tatianamac
Copy link
Collaborator

tatianamac commented Jun 10, 2020

Overview

Deque just released axe linter.
I have done no preliminary research on this, but thought it may be helpful to consider for our set up.

To be explicit, I never see linter tools or automated tests as a replacement for manual user testing in accessibility. Instead, I see them as helpers.

I wanted to bring this up in the context of the other open PRs and issues revolving around our linting set up (I can't tell which of these are open/closed, but probably helpful for context:

Needs

Ideally I'd like to:

  • Decide whether or not we will use axe Linter
  • Decide whether or not we use a link linter (like Linkinator)
  • Address and close any open PRs/issues around tests/linting
  • Include notes in our documentation

I think this task is probably best suited for someone with someone who has some familiarity with our database, though it is not necessary. Experience in this sort of area for set up would be great.

cc: @greatislander @ovlb for thoughts!

@tatianamac tatianamac added help wanted Good for external contributors Type · DX Affects developer experience labels Jun 10, 2020
@tatianamac tatianamac changed the title 🧹 Consider installing axe Linter 🧹 Finalise linting; consider axe Linting Jun 10, 2020
@greatislander
Copy link
Contributor

Hi @tatianamac! I agree it would be good to try the new axe-linter. I didn’t get anywhere with running axe-cli in GitHub Actions although I’m sure it is possible.

@tatianamac tatianamac changed the title 🧹 Finalise linting; consider axe Linting 🧹 Consolidate/finalise linting; consider axe Linting and Linkinator Jun 10, 2020
@bnb
Copy link
Contributor

bnb commented Jun 10, 2020

The axe Linter GitHub Action looks pretty nice. As you mentioned, not a replacement but it's a nice gate to have for shipping.

@ovlb
Copy link
Collaborator

ovlb commented Jun 12, 2020

Agreed with axe linter. @bnb, @greatislander Does one of you feel comfortable setting this up?

Linkinator looks interesting. We should give it a try!

@bnb
Copy link
Contributor

bnb commented Jun 17, 2020

axe Linter would need to be set up by a repo owner since it's a GitHub app. Should be just a few clicks, though!

@tatianamac
Copy link
Collaborator Author

Oh, it's a moot point as it's only free for personal open source projects, not OSS that are on organisation accounts. 😅😭

@greatislander
Copy link
Contributor

Dang.

@wuz
Copy link
Contributor

wuz commented Jul 1, 2020

@tatianamac I'd like to tackle adding some accessibility linting to the project with Github actions (I'm planning to do a stream tomorrow over this). Is that something you still are interested in, even though axe linter won't work?

@tatianamac
Copy link
Collaborator Author

@wuz Yes! I'd love that! Especially if it helps us to streamline/close this linting issue and reincorporate it back into our documentation. ✨ I'm going to assign this to you and mark it as in-progress (we can take it off if you change your mind).

@tatianamac tatianamac added Status · In Progress and removed help wanted Good for external contributors labels Jul 1, 2020
@wuz
Copy link
Contributor

wuz commented Jul 2, 2020

Awesome thanks!

@tatianamac tatianamac linked a pull request Aug 26, 2020 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Status · In Progress Type · DX Affects developer experience
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants