Skip to content

Commit

Permalink
lint: Build project in lint job to check imports
Browse files Browse the repository at this point in the history
  • Loading branch information
bertschneider committed Oct 5, 2023
1 parent 25618d2 commit 0f8789c
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 0 deletions.
1 change: 1 addition & 0 deletions .eslintrc.json
Original file line number Diff line number Diff line change
Expand Up @@ -24,6 +24,7 @@
"header/header": [2, ".header.js"],
"import/first": "error",
"import/newline-after-import": "error",
"import/no-unresolved": "warn",
"import/order": [
"error",
{
Expand Down
3 changes: 3 additions & 0 deletions .github/workflows/run-all-tests.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -26,6 +26,9 @@ jobs:
sudo apt-get install clang-tidy-16
sudo rm -f /usr/bin/clang-tidy
sudo ln -s /usr/bin/clang-tidy-16 /usr/bin/clang-tidy
- name: build project
# Build project so that imports can be checked during linting
run: npm run build
- name: build fuzzer
# Build the native addon so that CMake generates compile_commands.json that is needed by clang-tidy
run: npm run build --workspace=@jazzer.js/fuzzer
Expand Down

0 comments on commit 0f8789c

Please sign in to comment.