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

C++: Update test after extractor changes #18334

Merged
merged 1 commit into from
Dec 20, 2024

Conversation

jketema
Copy link
Contributor

@jketema jketema commented Dec 19, 2024

Pull Request checklist

All query authors

Internal query authors only

  • Autofixes generated based on these changes are valid, only needed if this PR makes significant changes to .ql, .qll, or .qhelp files. See the documentation (internal access required).
  • Changes are validated at scale (internal access required).
  • Adding a new query? Consider also adding the query to autofix.

@jketema jketema added the depends on internal PR This PR should only be merged in sync with an internal Semmle PR label Dec 19, 2024
@github-actions github-actions bot added the C++ label Dec 19, 2024
@jketema jketema force-pushed the template-parameters-1 branch from 8414ef9 to 6f9968d Compare December 20, 2024 07:28
@jketema jketema marked this pull request as ready for review December 20, 2024 08:18
@Copilot Copilot bot review requested due to automatic review settings December 20, 2024 08:18
@jketema jketema requested a review from a team as a code owner December 20, 2024 08:18

Choose a reason for hiding this comment

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

Copilot wasn't able to review any files in this pull request.

Files not reviewed (1)
  • cpp/ql/test/library-tests/unspecified_type/types/unspecified_type.expected: Language not supported

Tip: Copilot code review supports C#, Go, Java, JavaScript, Markdown, Python, Ruby and TypeScript, with more languages coming soon. Learn more

@jketema jketema merged commit 46e9d0a into github:main Dec 20, 2024
8 of 9 checks passed
@jketema jketema deleted the template-parameters-1 branch December 20, 2024 16:08
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.

2 participants