Skip to content

Re-instantiate Pkg registry in Documentation.yml#863

Closed
abhro wants to merge 1 commit into
SciML:masterfrom
abhro:patch-1
Closed

Re-instantiate Pkg registry in Documentation.yml#863
abhro wants to merge 1 commit into
SciML:masterfrom
abhro:patch-1

Conversation

@abhro

@abhro abhro commented May 28, 2026

Copy link
Copy Markdown
Contributor

Checklist

  • Appropriate tests were added
  • Any code changes were done in a way that does not break public API
  • All documentation related to code changes were updated
  • The new code follows the contributor guidelines, in particular the SciML Style Guide and COLPRAC.
  • Any new documentation only uses public API

Additional context

Maybe that will fix the GitHub workflow's problem on resolving versions when building docs (https://github.com/SciML/DiffEqDocs.jl/actions/runs/26536151206/job/78165442948). First waiting on the CI checks to see if it even works, because on a local machine I was unable to reproduce the error.

@abhro abhro marked this pull request as ready for review May 28, 2026 14:48
@abhro abhro closed this May 28, 2026
@abhro abhro reopened this May 28, 2026
@abhro

abhro commented May 28, 2026

Copy link
Copy Markdown
Contributor Author

So that worked, but now I'm concerned that it may be overwriting global registries on the CI machine? (Because Pkg registries on Julia aren't environment specific.) But on the other hand, if the machine is only for CI, then any job would probably just re-add a registry or whatever so we don't need to worry about that.

@abhro

abhro commented May 28, 2026

Copy link
Copy Markdown
Contributor Author

Well, the documentation build fails in a different way now, so this PR succeeded in its aims! 🥰

@abhro abhro changed the title Add ]registry update to Documentation.yml Re-instantiate Pkg registry in Documentation.yml May 28, 2026
@abhro abhro force-pushed the patch-1 branch 3 times, most recently from 9a18a1d to a3429dc Compare May 28, 2026 16:04
@ChrisRackauckas

Copy link
Copy Markdown
Member

I don't think this is the solution. The runner should just get its registry update unblocked.

@abhro

abhro commented May 29, 2026

Copy link
Copy Markdown
Contributor Author

I'm not sure what you mean by the "registry update unblocked"?

@ChrisRackauckas

Copy link
Copy Markdown
Member

We have the new runners online, but just need the pool to be opted into here.

@ChrisRackauckas

Copy link
Copy Markdown
Member

With the new runner pool we shouldn't need to do this.

@abhro abhro deleted the patch-1 branch June 2, 2026 11:55
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.

2 participants