Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly
Appearance settings

Add query suite inclusion tests for cpp, python #19390

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.

Sign up for GitHub

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

Jump to bottom
Merged
tamasvajk merged 2 commits into from
Apr 29, 2025
Merged

Add query suite inclusion tests for cpp, python #19390

tamasvajk merged 2 commits into from
Apr 29, 2025

Conversation

@tamasvajk
Copy link
Contributor

@tamasvajk tamasvajk commented Apr 28, 2025 •
edited
Loading

This pull request adds query suite integration tests for cpp and python. Some changes are also required in the private repo, because integration tests for these two languages were previously not added in this public repo.

Additionally, the Actions query suite integration test is fixed.

@tamasvajk tamasvajk marked this pull request as ready for review April 28, 2025 07:21
Copilot AI review requested due to automatic review settings April 28, 2025 07:21
@tamasvajk tamasvajk requested review from a team as code owners April 28, 2025 07:21
Copilot AI reviewed Apr 28, 2025
View reviewed changes
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@tamasvajk tamasvajk requested a review from a team as a code owner April 28, 2025 08:33
@github-actions github-actions bot added the Actions Analysis of GitHub Actions label Apr 28, 2025
@tamasvajk tamasvajk force-pushed the test/cpp-python-query-suites branch from ff36432 to 3a0759b Compare April 28, 2025 08:43
jketema
jketema approved these changes Apr 28, 2025
View reviewed changes
Copy link
Contributor

@jketema jketema left a comment

Choose a reason for hiding this comment

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

LGTM

@tamasvajk tamasvajk force-pushed the test/cpp-python-query-suites branch from 3a0759b to e73031c Compare April 29, 2025 06:45
@github-actions github-actions bot removed the Actions Analysis of GitHub Actions label Apr 29, 2025
@tamasvajk tamasvajk merged commit 1694050 into github:main Apr 29, 2025
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@michaelnebel michaelnebel michaelnebel left review comments

Copilot code review Copilot Copilot left review comments

@jketema jketema jketema approved these changes

Assignees

No one assigned

Labels

C++ Python

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@tamasvajk @michaelnebel @jketema
Morty Proxy This is a proxified and sanitized view of the page, visit original site.