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

Cleanup MemoryRef tests #54681

Merged

Conversation

oscardssmith
Copy link
Member

@oscardssmith oscardssmith commented Jun 5, 2024

aka revert revert #54647. Sorry @Keno for merging with broken tests. Between unrelated failures on windows CI upload and the fact that the errors were all truncated, I missed it.

@oscardssmith oscardssmith added the bugfix This change fixes an existing bug label Jun 5, 2024
@giordano giordano changed the title os/fix-cleanup-MemoryRef-tests Cleanup MemoryRef tests Jun 5, 2024
@giordano giordano added the re-land This relands a PR that was previously merged but was later reverted. label Jun 5, 2024
@oscardssmith oscardssmith force-pushed the os/fix-cleanup-MemoryRef-tests branch from ed99cdc to 3b1cacc Compare June 5, 2024 12:54
@oscardssmith oscardssmith merged commit a5d6b50 into JuliaLang:master Jun 5, 2024
7 of 8 checks passed
@oscardssmith oscardssmith deleted the os/fix-cleanup-MemoryRef-tests branch June 5, 2024 17:26
aviatesk added a commit that referenced this pull request Jun 14, 2024
oscardssmith pushed a commit that referenced this pull request Jun 14, 2024
KristofferC pushed a commit that referenced this pull request Jun 18, 2024
KristofferC pushed a commit that referenced this pull request Jun 18, 2024
KristofferC added a commit that referenced this pull request Jun 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bugfix This change fixes an existing bug re-land This relands a PR that was previously merged but was later reverted.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants