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.

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

Verify error reporting from the CI-weekly run #6060

Merged
merged 3 commits into from
Apr 13, 2023
Merged

Verify error reporting from the CI-weekly run #6060

merged 3 commits into from
Apr 13, 2023

Conversation

pavoljuhas
Copy link
Collaborator

Intentionally fail one tests to verify error report from CI-weekly workflow.

Related to #6037

@pavoljuhas
Verify error reporting from the CI-weekly run
d321f86 
Intentionally fail one tests to verify error report from CI-weekly workflow.

Related to quantumlib#6037
@pavoljuhas pavoljuhas requested review from a team, vtomole and cduck as code owners April 12, 2023 22:24
@pavoljuhas pavoljuhas requested a review from maffoo April 12, 2023 22:24
@CirqBot CirqBot added the Size: XS <10 lines changed label Apr 12, 2023
@pavoljuhas
CI wake up - no code change
7b77cbf 
Maybe pytest needs a different seed to pass Pytest Windows 3.7
@pavoljuhas
Merge branch 'master' into ci-weekly-check-error-reporting
8a7987d
@pavoljuhas pavoljuhas merged commit 1445f12 into quantumlib:master Apr 13, 2023
@pavoljuhas pavoljuhas deleted the ci-weekly-check-error-reporting branch April 13, 2023 18:24
@pavoljuhas pavoljuhas mentioned this pull request Apr 14, 2023
Merged
pavoljuhas added a commit to pavoljuhas/Cirq that referenced this pull request Apr 14, 2023
@pavoljuhas
Restore weekly cadence for the CI-weekly workflow
0cf6d02 
Also remove the intentional test failure as
it was correctly reported by the CI.

Finalizes quantumlib#6037
Reverts quantumlib#6060
pavoljuhas added a commit that referenced this pull request Apr 14, 2023
@pavoljuhas
Restore weekly cadence for the CI-weekly workflow (#6066)
79286a1 
And also remove the intentional test failure as it was correctly reported by the CI.

Finalizes #6037
Reverts #6060
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@verult verult verult approved these changes

@vtomole vtomole Awaiting requested review from vtomole

@cduck cduck Awaiting requested review from cduck

@maffoo maffoo Awaiting requested review from maffoo maffoo was automatically assigned from quantumlib/cirq-maintainers

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.
3 participants
@pavoljuhas @verult @CirqBot