Skip to content

Commit

Permalink
[Deps] pin aria-query and axobject-query, add ls-engines test t…
Browse files Browse the repository at this point in the history
…o CI

Specifically, v5.2 of aria-query and v3.2 of axobject-query require node 6 via the dequal dependency, and neither have an engines declaration. This is a breaking change

See A11yance/aria-query#497 (comment)
  • Loading branch information
ljharb committed Jun 19, 2024
1 parent 0a98ad8 commit 32fd82c
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 3 deletions.
7 changes: 6 additions & 1 deletion .github/workflows/node-pretest.yml
Original file line number Diff line number Diff line change
Expand Up @@ -49,4 +49,9 @@ jobs:
with:
node-version: 'lts/*'
skip-ls-check: true
- run: npm run test:examples
- run: npm run test:examples

engines:
runs-on: ubuntu-latest
steps:
- uses: ljharb/actions/node/engines@main
4 changes: 2 additions & 2 deletions package.json
Original file line number Diff line number Diff line change
Expand Up @@ -79,12 +79,12 @@
},
"license": "MIT",
"dependencies": {
"aria-query": "^5.3.0",
"aria-query": "~5.1.3",
"array-includes": "^3.1.8",
"array.prototype.flatmap": "^1.3.2",
"ast-types-flow": "^0.0.8",
"axe-core": "^4.9.1",
"axobject-query": "^3.2.1",
"axobject-query": "~3.1.1",
"damerau-levenshtein": "^1.0.8",
"emoji-regex": "^9.2.2",
"es-iterator-helpers": "^1.0.19",
Expand Down

0 comments on commit 32fd82c

Please sign in to comment.