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 unused warnings filter for pytest #6186

Merged
merged 3 commits into from Jul 7, 2023

Conversation

pavoljuhas
Copy link
Collaborator

@pavoljuhas pavoljuhas commented Jul 6, 2023

Verify that warning filter rules are not matched and then remove them.

Clean up after #4737 which does not bite anymore

Change warnings filter action to error so they cause pytest failure.
pytest passes without catching any of specified warnings.
@pavoljuhas pavoljuhas marked this pull request as ready for review July 6, 2023 18:29
@pavoljuhas pavoljuhas requested review from a team, vtomole and cduck as code owners July 6, 2023 18:29
@pavoljuhas pavoljuhas enabled auto-merge (squash) July 7, 2023 01:12
@CirqBot CirqBot added the size: S 10< lines changed <50 label Jul 7, 2023
@pavoljuhas pavoljuhas merged commit 5d3e681 into quantumlib:master Jul 7, 2023
30 checks passed
@pavoljuhas pavoljuhas deleted the drop-old-warnings-filter branch July 7, 2023 02:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size: S 10< lines changed <50
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants