Skip to content

C++: Update expected test results - #22379

Merged
jketema merged 1 commit into
github:mainfrom
jketema:jketema/constant-filter
Aug 21, 2026
Merged

C++: Update expected test results#22379
jketema merged 1 commit into
github:mainfrom
jketema:jketema/constant-filter

Conversation

@jketema

@jketema jketema commented Aug 19, 2026

Copy link
Copy Markdown
Contributor

No description provided.

@github-actions github-actions Bot added the C++ label Aug 19, 2026
@jketema jketema added depends on internal PR This PR should only be merged in sync with an internal Semmle PR and removed C++ labels Aug 19, 2026
@jketema
jketema marked this pull request as ready for review August 19, 2026 11:18
@jketema
jketema requested a review from a team as a code owner August 19, 2026 11:18
Copilot AI balanced review requested due to automatic review settings August 19, 2026 11:18

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates C++ syntax-zoo expected results after resolved data-flow consistency findings.

Changes:

  • Removes obsolete builtin.c expected diagnostics.
Show a summary per file
File Description
cpp/ql/test/library-tests/syntax-zoo/dataflow-consistency.expected Updates expected consistency results.

Review details

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

  • Files reviewed: 1/1 changed files
  • Comments generated: 0
  • Review effort level: Balanced

@jketema
jketema marked this pull request as draft August 19, 2026 12:19
@jketema
jketema force-pushed the jketema/constant-filter branch from 6d0aa81 to 3de9880 Compare August 21, 2026 13:59
@jketema
jketema marked this pull request as ready for review August 21, 2026 14:00
@github-actions github-actions Bot added the C++ label Aug 21, 2026
@jketema
jketema merged commit 5eac06d into github:main Aug 21, 2026
9 of 10 checks passed
@jketema
jketema deleted the jketema/constant-filter branch August 21, 2026 17:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

C++ depends on internal PR This PR should only be merged in sync with an internal Semmle PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants