-
Notifications
You must be signed in to change notification settings - Fork 298
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
Created Github Workflow for Accessibility Testing (Using Axe) #879
Changes from 2 commits
29f2f43
789964f
cf1c0eb
e6ac640
6ff61db
f6d2c51
e1e8c16
5f8f5c6
9764a68
7e1176a
f5c7bf1
d975ac4
3bce8eb
94a29e9
54e1f04
8753e5e
0eccd5b
e1de511
f7a048e
61c5b79
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change | ||
---|---|---|---|---|
@@ -0,0 +1,22 @@ | ||||
name: axe | ||||
on: | ||||
push: | ||||
branches: [master] | ||||
pull_request: | ||||
branches: [master] | ||||
jobs: | ||||
axe: | ||||
runs-on: ubuntu-latest | ||||
steps: | ||||
- uses: actions/checkout@v2 | ||||
- name: Use Node.js 16.x, 18.x | ||||
uses: actions/setup-node@v1 | ||||
with: | ||||
node-version: 16.x, 18.x | ||||
- run: npm ci | ||||
- run: npm run build --if-present | ||||
- run: npm start & npx wait-on http://localhost:4010 | ||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. We don't have a start script! Line 6 in dae72cf
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. It appears so. To add a start script, I need to know the entry point to the application. From my research, it is typically specified in the main field of the package.json file as "index.js" or "app.js" or "server.js". But I don't see it. I've scoured the entire repo and I can't find anything remotely close to it. I was hoping you could point me in the right direction, please. |
||||
- name: Run axe | ||||
run: | | ||||
npm install -g @axe-core/cli | ||||
axe http://localhost:4010 --exit |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's include develop too!!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done