Skip to content
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

Merge main into releases/v2 #1570

Merged
merged 34 commits into from
Mar 10, 2023
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
34 commits
Select commit Hold shift + click to select a range
bbe8d37
Ensure qlconfig file is created when config parsing in cli is on
aeisenberg Feb 7, 2023
e2f72f1
Merge remote-tracking branch 'upstream/main' into aeisenberg/qlconfig…
aeisenberg Feb 9, 2023
3c81243
Apply suggestions from code review
aeisenberg Feb 9, 2023
5492b7d
Add tests for `generateRegistries` with an existing CODEQL_REGISTRIES…
aeisenberg Feb 9, 2023
736263f
Update src/codeql.ts
aeisenberg Feb 17, 2023
652709d
Update changelog and version after v2.2.5
invalid-email-address Feb 24, 2023
cf1855a
Fix workflow to update dependencies
henrymercer Feb 24, 2023
8f19113
Merge branch 'main' into aeisenberg/qlconfig-in-cli
aeisenberg Feb 27, 2023
041757f
Merge pull request #1549 from github/henrymercer/fix-update-dependencies
henrymercer Feb 27, 2023
36a249f
Merge branch 'main' into mergeback/v2.2.5-to-main-32dc4993
henrymercer Feb 27, 2023
18ff14b
Update checked-in dependencies
invalid-email-address Feb 27, 2023
903be79
Merge pull request #1548 from github/mergeback/v2.2.5-to-main-32dc4993
henrymercer Feb 27, 2023
98d24e5
Address comments from PR
aeisenberg Feb 27, 2023
a589d40
Merge pull request #1527 from github/aeisenberg/qlconfig-in-cli
aeisenberg Feb 27, 2023
e812e63
Update bundle to 2.12.4
dbartol Mar 6, 2023
5e4af3a
Update bundle to 2.12.4
dbartol Mar 6, 2023
a6d3a44
Avoid writing `user-config.yaml` file while running tests
henrymercer Mar 6, 2023
100bd7b
Avoid passing an undefined qlconfig arg
henrymercer Mar 6, 2023
6f079be
Bump `setup-swift` and re-configure dependabot (#1560)
angelapwen Mar 6, 2023
e5ade42
Merge branch 'main' into dbartol/bundle-2.12.4
henrymercer Mar 6, 2023
abf1cea
Bump glob from 8.0.1 to 9.2.1
dependabot[bot] Mar 6, 2023
7806af3
Update checked-in dependencies
invalid-email-address Mar 6, 2023
eb40427
Fix name of qlconfig file argument
henrymercer Mar 6, 2023
6ef6e50
Merge pull request #1563 from github/dependabot/npm_and_yarn/glob-9.2.1
henrymercer Mar 6, 2023
8340258
Avoid writing `user-config.yaml` file while running tests
henrymercer Mar 6, 2023
4366485
Avoid passing an undefined qlconfig arg
henrymercer Mar 6, 2023
c310f09
Fix name of qlconfig file argument
henrymercer Mar 6, 2023
e4b846c
Merge pull request #1564 from github/aeisenberg/qlconfig-file
aeisenberg Mar 7, 2023
0c27d0d
Add default values to feature flags
aeisenberg Mar 8, 2023
a3cf964
Add `security-experimental` to `codeql-config.yml` (#1566)
angelapwen Mar 8, 2023
f13b180
Merge pull request #1567 from github/aeisenberg/config-parsing-ghes
aeisenberg Mar 8, 2023
d47d4c8
Merge branch 'main' into dbartol/bundle-2.12.4
dbartol Mar 9, 2023
e12a2ec
Merge pull request #1561 from github/dbartol/bundle-2.12.4
dbartol Mar 9, 2023
74cbab4
Update changelog for v2.2.6
invalid-email-address Mar 10, 2023
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
1 change: 1 addition & 0 deletions .github/codeql/codeql-config.yml
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,7 @@ queries:
# we include both even though one is a superset of the
# other, because we're testing the parsing logic and
# that the suites exist in the codeql bundle.
- uses: security-experimental
- uses: security-extended
- uses: security-and-quality
paths-ignore:
Expand Down
4 changes: 4 additions & 0 deletions .github/dependabot.yml
Original file line number Diff line number Diff line change
Expand Up @@ -15,3 +15,7 @@ updates:
directory: "/"
schedule:
interval: weekly
- package-ecosystem: github-actions
directory: "/.github/setup-swift/" # All subdirectories outside of "/.github/workflows" must be explicitly included.
schedule:
interval: weekly
2 changes: 1 addition & 1 deletion .github/setup-swift/action.yml
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,7 @@ runs:
VERSION="5.7.0"
fi
echo "version=$VERSION" | tee -a $GITHUB_OUTPUT
- uses: swift-actions/setup-swift@194625b58a582570f61cc707c3b558086c26b723
- uses: swift-actions/setup-swift@da0e3e04b5e3e15dbc3861bd835ad9f0afe56296 # Please update the corresponding SHA in the CLI's CodeQL Action Integration Test.
if: "(runner.os != 'Windows') && (matrix.version == 'cached' || matrix.version == 'latest' || matrix.version == 'nightly-latest')"
with:
swift-version: "${{steps.get_swift_version.outputs.version}}"
42 changes: 42 additions & 0 deletions .github/workflows/__init-with-registries.yml

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

10 changes: 1 addition & 9 deletions .github/workflows/script/check-node-modules.sh
Original file line number Diff line number Diff line change
Expand Up @@ -7,17 +7,9 @@ if [ ! -z "$(git status --porcelain)" ]; then
>&2 echo "Failed: Repo should be clean before testing!"
exit 1
fi
# When updating this, make sure to update the npm version in
# `.github/workflows/update-dependencies.yml` too.
sudo npm install --force -g npm@9.2.0

# clean the npm cache to ensure we don't have any files owned by root
sudo npm cache clean --force
"$(dirname "$0")/update-node-modules.sh" check-only

# Reinstall modules and then clean to remove absolute paths
# Use 'npm ci' instead of 'npm install' as this is intended to be reproducible
npm ci
npm run removeNPMAbsolutePaths
# Check that repo is still clean
if [ ! -z "$(git status --porcelain)" ]; then
# If we get a fail here then the PR needs attention
Expand Down
18 changes: 18 additions & 0 deletions .github/workflows/script/update-node-modules.sh
Original file line number Diff line number Diff line change
@@ -0,0 +1,18 @@
if [ "$1" != "update" && "$1" != "check-only" ]; then
>&2 echo "Failed: Invalid argument. Must be 'update' or 'check-only'"
exit 1
fi

sudo npm install --force -g npm@9.2.0

# clean the npm cache to ensure we don't have any files owned by root
sudo npm cache clean --force

if [ "$1" = "update" ]; then
npm install
fi

# Reinstall modules and then clean to remove absolute paths
# Use 'npm ci' instead of 'npm install' as this is intended to be reproducible
npm ci
npm run removeNPMAbsolutePaths
7 changes: 1 addition & 6 deletions .github/workflows/update-dependencies.yml
Original file line number Diff line number Diff line change
Expand Up @@ -27,12 +27,7 @@ jobs:
run: |
git fetch origin "$BRANCH" --depth=1
git checkout "origin/$BRANCH"
# When updating this, make sure to update the npm version in
# `.github/workflows/script/check-node-modules.sh` too.
sudo npm install --force -g npm@9.2.0
npm install
npm ci
npm run removeNPMAbsolutePaths
.github/workflows/script/update-node-modules.sh update
if [ ! -z "$(git status --porcelain)" ]; then
git config --global user.email "github-actions@github.com"
git config --global user.name "github-actions[bot]"
Expand Down
4 changes: 4 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
@@ -1,5 +1,9 @@
# CodeQL Action Changelog

## 2.2.6 - 10 Mar 2023

- Update default CodeQL bundle version to 2.12.4.

## 2.2.5 - 24 Feb 2023

- Update default CodeQL bundle version to 2.12.3. [#1543](https://github.com/github/codeql-action/pull/1543)
Expand Down
28 changes: 18 additions & 10 deletions lib/codeql.js

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

2 changes: 1 addition & 1 deletion lib/codeql.js.map

Large diffs are not rendered by default.

75 changes: 59 additions & 16 deletions lib/codeql.test.js

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

2 changes: 1 addition & 1 deletion lib/codeql.test.js.map

Large diffs are not rendered by default.