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

Remove stray debug prints from tests #842

Merged
merged 1 commit into from
Mar 21, 2023

Conversation

mtreinish
Copy link
Member

This commit removes some stray debug print messages that were in some test files. This was causing spurious output that wasn't actionable during test runs both locally and in CI.

This commit removes some stray debug print messages that were in some
test files. This was causing spurious output that wasn't actionable
during test runs both locally and in CI.
@coveralls
Copy link

Pull Request Test Coverage Report for Build 4473388980

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • 3 unchanged lines in 1 file lost coverage.
  • Overall coverage decreased (-0.04%) to 97.114%

Files with Coverage Reduction New Missed Lines %
src/iterators.rs 3 86.74%
Totals Coverage Status
Change from base Build 4472444201: -0.04%
Covered Lines: 13896
Relevant Lines: 14309

💛 - Coveralls

Copy link
Collaborator

@IvanIsCoding IvanIsCoding left a comment

Choose a reason for hiding this comment

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

LGTM, I agree that those were left by accident

@IvanIsCoding IvanIsCoding added the automerge Queue a approved PR for merging label Mar 21, 2023
@mergify mergify bot merged commit e65185c into Qiskit:main Mar 21, 2023
@mtreinish mtreinish deleted the remove-prints-tests branch March 22, 2023 14:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automerge Queue a approved PR for merging
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants