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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Shuffle the tests, just in case #2272

Merged
merged 2 commits into from Mar 23, 2023
Merged

Shuffle the tests, just in case #2272

merged 2 commits into from Mar 23, 2023

Conversation

jvoisin
Copy link
Collaborator

@jvoisin jvoisin commented Mar 21, 2023

No description provided.

@github-actions
Copy link

Download the artifacts for this pull request:

@jvoisin jvoisin requested a review from deluan March 21, 2023 19:57
@deluan
Copy link
Member

deluan commented Mar 22, 2023

Nice! I think we should also do this when running tests locally:

go test -race ./...

@deluan deluan merged commit 17e0cd5 into navidrome:master Mar 23, 2023
4 checks passed
@jvoisin jvoisin deleted the shuffle branch March 23, 2023 11:48
@github-actions
Copy link

This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jul 22, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants