Skip to content

Commit

Permalink
🔧👷 properly exclude build directory from CodeQL (#242)
Browse files Browse the repository at this point in the history
## Description

This small PR adjusts the CodeQL config to ignore the `build` folder.
Since the introduction of using `FetchContent`, the build folder also
contains a `_deps` subfolder with all the fetched dependencies. These
might (and in the case of QECC here, will) trigger CodeQL warnings we
have no control over.
This PR should reduce the number of warnings flagged for the project.

## Checklist:

<!---
This checklist serves as a reminder of a couple of things that ensure
your pull request will be merged swiftly.
-->

- [x] The pull request only contains commits that are related to it.
- [x] I have added appropriate tests and documentation.
- [x] I have made sure that all CI jobs on GitHub pass.
- [x] The pull request introduces no new warnings and follows the
project's style guidelines.

Signed-off-by: burgholzer <[email protected]>
  • Loading branch information
burgholzer authored Jul 17, 2024
1 parent d084afc commit 15b74be
Showing 1 changed file with 1 addition and 0 deletions.
1 change: 1 addition & 0 deletions .github/workflows/reusable-code-ql.yml
Original file line number Diff line number Diff line change
Expand Up @@ -65,6 +65,7 @@ jobs:
with:
patterns: |
-**/extern/**
-**/build/**
input: sarif-results/${{ matrix.language }}.sarif
output: sarif-results/${{ matrix.language }}.sarif

Expand Down

0 comments on commit 15b74be

Please sign in to comment.