Repository navigation

Include diagnostics in failed SARIF if they were created before a database was initialised - #4223

Merged

mbg merged 21 commits into

mainfrom

mbg/diagnostics/failed-sarif

Oct 22, 2026

Merged

Include diagnostics in failed SARIF if they were created before a database was initialised#4223
mbg merged 21 commits into
mainfrom
mbg/diagnostics/failed-sarif

Commits on Oct 20, 2026
- Refactor `writeDiagnosticFile` out of `writeDiagnostic`
  mbg
  authored
  56e0d42
- Write diagnostics to temporary location when database isn't ready
  mbg
  authored
  f422d8a
- Make diagnostics less dependent on global state
  mbg
  authored
  67f2591
- Use temporary diagnostic dir in `generateFailedSarif`
  mbg
  authored
  64723df
- Refactor `writeTemporaryDiagnosticFile` out of `add[NoLanguage]Diagnostic`
  mbg
  authored
  581a0e1
- Omit `--sarif-codescanning-config` if the config file doesn't exist
  mbg
  authored
  ba569ac
- Add e2e test
  mbg
  authored
  b963054
Commits on Oct 21, 2026
- Use same sub-path in `getDiagnosticsPath` for no language case
  mbg
  committed
  8357829
- Merge remote-tracking branch 'origin/main' into mbg/diagnostics/failed-sarif
  mbg
  committed
  d7bad7d
- Always check whether config file exists
  mbg
  committed
  98b8e51
- Don't include diagnostics if FF is off
  mbg
  committed
  4707994
- Rename env var to `DIAGNOSTICS_DIR`
  mbg
  committed
  67cd8c1
- Update skipped `analyze` step comment
  mbg
  committed
  a83f6f5
- Clarify comment that explains how the test works
  mbg
  committed
  c3f2a34
Commits on Oct 22, 2026
- Improve logging and error handling in `logTemporaryDiagnostics`
  mbg
  committed
  26f5c5e
- Only warn for missing config if `init` was successful
  mbg
  committed
  308d941
- Clean up path functions in `diagnostics.ts` and add some tests
  mbg
  committed
  b79966a
- Merge branch 'main' into mbg/diagnostics/failed-sarif
  mbg
  committed
  2d0d1c1
- Merge branch 'main' into mbg/diagnostics/failed-sarif
  mbg
  committed
  ac509b6
- Improve wording to address review feedback
  mbg
  committed
  87da5f3
- Merge remote-tracking branch 'origin/main' into mbg/diagnostics/failed-sarif
  mbg
  committed
  5196bae

Include diagnostics in failed SARIF if they were created before a database was initialised by mbg · Pull Request #4223 · github/codeql-action · GitHub

Repository navigation

Include diagnostics in failed SARIF if they were created before a database was initialised - #4223

Include diagnostics in failed SARIF if they were created before a database was initialised#4223mbg merged 21 commits intomaingithub/codeql-action:mainfrom mbg/diagnostics/failed-sarifgithub/codeql-action:mbg/diagnostics/failed-sarifCopy head branch name to clipboard

Commits

Commits on Oct 20, 2026

Commits on Oct 21, 2026

Commits on Oct 22, 2026

Include diagnostics in failed SARIF if they were created before a database was initialised#4223
mbg merged 21 commits into
mainfrom
mbg/diagnostics/failed-sarif