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

[3.12] PackageTests/NewUpdate: fix skipping flaky tests #10045

Merged
merged 1 commit into from
May 23, 2024

Conversation

bacchanalia
Copy link
Collaborator

@bacchanalia bacchanalia commented May 23, 2024

RejectFutureIndexStates and UpdateIndexState are marked "skip", but it's under withRemoteRepo, which causes flakiness before skip is called.

backport of #10035

@bacchanalia bacchanalia force-pushed the zz/fix-flaky-tests-3.12 branch from 5515248 to cc85b3b Compare May 23, 2024 03:31
@ulysses4ever ulysses4ever added merge me Tell Mergify Bot to merge backport labels May 23, 2024
@Mikolaj
Copy link
Member

Mikolaj commented May 23, 2024

If nobody objects, I'm going to expedite the merge to shorten the period we are getting the spurious CI errors.

@ulysses4ever
Copy link
Collaborator

@Mikolaj backports are exempt from the two-days delay anyway. The bot shall merge it once CI passes

@ulysses4ever
Copy link
Collaborator

@mergify refresh

Copy link
Contributor

mergify bot commented May 23, 2024

refresh

✅ Pull request refreshed

@Mikolaj Mikolaj force-pushed the zz/fix-flaky-tests-3.12 branch from cc85b3b to c0ea4ba Compare May 23, 2024 15:08
@ulysses4ever
Copy link
Collaborator

@ulysses4ever ulysses4ever force-pushed the zz/fix-flaky-tests-3.12 branch from c0ea4ba to 82bf23b Compare May 23, 2024 18:35
RejectFutureIndexStates and UpdateIndexState are marked "skip", but it's
under withRemoteRepo, which causes flakiness before skip is called.
@ulysses4ever ulysses4ever force-pushed the zz/fix-flaky-tests-3.12 branch from 82bf23b to 643cd64 Compare May 23, 2024 19:12
@mergify mergify bot merged commit 5cd189c into haskell:3.12 May 23, 2024
48 checks passed
ulysses4ever pushed a commit to ulysses4ever/cabal that referenced this pull request Jun 3, 2024
…3.12

[3.12] PackageTests/NewUpdate: fix skipping flaky tests
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport merge me Tell Mergify Bot to merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants