Skip to content

Fix integration tests and benchmarks#2713

Merged
diego-plan9 merged 3 commits intoQiskit:mainfrom
diego-plan9:fix/integration-tests-and-benchmarks
Apr 1, 2026
Merged

Fix integration tests and benchmarks#2713
diego-plan9 merged 3 commits intoQiskit:mainfrom
diego-plan9:fix/integration-tests-and-benchmarks

Conversation

@diego-plan9
Copy link
Copy Markdown
Member

Summary

Fixes for a couple of oversights from #2607 and #2708:

  • pass the url to one of the proxy tests, as otherwise the staging run would fail due to using production's IAM
  • revise the skipping of the benchmarks (reusing some decorator pieces), to ensure they are not always skipped in CI, as well as using seconds always in the output and take a bit of time off conforming to Refactor package extras #2707

Details and comments

Fixes N/A

@diego-plan9 diego-plan9 requested a review from rainerschoe April 1, 2026 10:38
Copy link
Copy Markdown
Collaborator

@rainerschoe rainerschoe left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks for the explanations.

@diego-plan9 diego-plan9 added this pull request to the merge queue Apr 1, 2026
Merged via the queue into Qiskit:main with commit 52107be Apr 1, 2026
17 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants