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

avoid private access to loaded_precompile #856

Merged
merged 3 commits into from
Oct 10, 2024
Merged

Conversation

vtjnash
Copy link
Collaborator

@vtjnash vtjnash commented Oct 5, 2024

No description provided.

src/loading.jl Outdated Show resolved Hide resolved
vtjnash added a commit to JuliaLang/julia that referenced this pull request Oct 7, 2024
If we expect this to be a public API
(https://github.com/timholy/Revise.jl for some reason is trying to
access this state), we should lock around it for consistency with the
other similar functions.

Needed for timholy/Revise.jl#856
@vtjnash vtjnash closed this Oct 8, 2024
@vtjnash vtjnash reopened this Oct 8, 2024
Co-authored-by: Cody Tapscott <[email protected]>
@topolarity
Copy link
Contributor

topolarity commented Oct 9, 2024

I believe the nightly failure is because nightly is not new enough (it's missing JuliaLang/julia#56027)

@KristofferC
Copy link
Collaborator

Weird it hasn't been updated in two days.

@vtjnash
Copy link
Collaborator Author

vtjnash commented Oct 9, 2024

I think Pkg is broken on v1.11, so the upload jobs have been failing for a couple days, if I read the logs right

@topolarity
Copy link
Contributor

topolarity commented Oct 9, 2024

I've tested master locally and it passes - Good to merge?

@Keno Keno merged commit 336e39c into master Oct 10, 2024
14 of 17 checks passed
@KristofferC KristofferC deleted the jn/maybe_loaded_precompile branch October 10, 2024 09:56
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.

4 participants