Skip to content

C++: Update expected test results - #22379

Draft
jketema wants to merge 1 commit into
github:mainfrom
jketema:jketema/constant-filter
Draft

C++: Update expected test results#22379
jketema wants to merge 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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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.

2 participants