Skip to content

Remove commented-out code #941

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
merged 2 commits into from
Nov 4, 2025
Merged

Remove commented-out code #941

merged 2 commits into from
Nov 4, 2025

Conversation

@mhucka
Copy link
Collaborator

@mhucka mhucka commented Nov 4, 2025

Not sure why this was commented out, but a Code Quality scan flagged it, so we may as well remove it.

@github-actions github-actions bot added the Size: XS <10 lines changed label Nov 4, 2025
gemini-code-assist[bot]
gemini-code-assist bot reviewed Nov 4, 2025
View reviewed changes
Copy link

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request removes commented-out code from the NoiseTrigger test helper class. This is a good code hygiene practice that improves readability and maintainability. The change is correct and I see no issues.

@mhucka mhucka enabled auto-merge November 4, 2025 19:10
@mhucka mhucka added this pull request to the merge queue Nov 4, 2025
Merged via the queue into quantumlib:main with commit 3af5e18 Nov 4, 2025
52 checks passed
@mhucka mhucka deleted the mh-trivial-comment-edit branch November 4, 2025 20:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@pavoljuhas pavoljuhas pavoljuhas approved these changes

@sergeisakov sergeisakov Awaiting requested review from sergeisakov

+1 more reviewer

@gemini-code-assist gemini-code-assist[bot] gemini-code-assist[bot] left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

Size: XS <10 lines changed

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@mhucka @pavoljuhas