Skip to content

Commit fa18c60

Browse files
authored
Merge pull request #672 from github/henrymercer/reenable-diagnostics-summaries
Re-enable diagnostics summaries in the output logs of the `analyze` action
2 parents 60484b2 + adc7ff6 commit fa18c60

4 files changed

Lines changed: 4 additions & 2 deletions

File tree

CHANGELOG.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@
22

33
## [UNRELEASED]
44

5-
No user facing changes.
5+
- Fix an issue where a summary of diagnostics information from CodeQL was not output to the logs of the `analyze` step of the Action. [#672](https://github.com/github/codeql-action/pull/672)
66

77
## 1.0.9 - 02 Aug 2021
88

lib/codeql.js

Lines changed: 1 addition & 0 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

lib/codeql.js.map

Lines changed: 1 addition & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

src/codeql.ts

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -754,6 +754,7 @@ function getCodeQLForCmd(cmd: string): CodeQL {
754754
"interpret-results",
755755
threadsFlag,
756756
"--format=sarif-latest",
757+
"--print-diagnostics-summary",
757758
"--print-metrics-summary",
758759
"--sarif-group-rules-by-pack",
759760
"-v",

0 commit comments

Comments
 (0)