Skip to content

Commit

Permalink
[INTERNAL] Fix missing peerDependency to acorn / Add CI check
Browse files Browse the repository at this point in the history
Due to a bug in npm, acorn-jsx can't find the correct version of acorn.

Downgrading to ava@3.11 solves the issue as only one acorn version (v7) is present.

Also adding an 'npm ls' check to detect such issues before they get merged into master.
  • Loading branch information
matz3 committed Sep 2, 2020
1 parent 95ac50f commit b48b4f6
Show file tree
Hide file tree
Showing 3 changed files with 101 additions and 30 deletions.
3 changes: 3 additions & 0 deletions azure-pipelines.yml
Original file line number Diff line number Diff line change
Expand Up @@ -36,6 +36,9 @@ steps:
- script: npm ci
displayName: Install Dependencies

- script: npm ls
displayName: Check for missing / extraneous Dependencies

- script: npm run test-azure
displayName: Run Tests

Expand Down
126 changes: 97 additions & 29 deletions package-lock.json

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion package.json
Original file line number Diff line number Diff line change
Expand Up @@ -124,7 +124,7 @@
"yazl": "^2.5.1"
},
"devDependencies": {
"ava": "^3.12.1",
"ava": "3.11.1",
"chai": "^4.1.2",
"chai-fs": "^2.0.0",
"chokidar-cli": "^2.1.0",
Expand Down

0 comments on commit b48b4f6

Please sign in to comment.