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

Allow skipping cloning of General registry #23

Closed
wants to merge 5 commits into from

Conversation

omus
Copy link
Contributor

@omus omus commented Jan 4, 2024

Depends on:

Allows for the clone of the General registry to be optional.

main.js Outdated Show resolved Hide resolved
@omus
Copy link
Contributor Author

omus commented Jan 4, 2024

I may need to rethink this. It seems that Julia doesn't add the General registry if a registry has already been added 🤔

ERROR: LoadError: expected package `StableRNGs [860ef19b]` to be registered

@omus
Copy link
Contributor Author

omus commented Jan 4, 2024

I'm going to approach this in a different way where each registry needs to be specified.

@omus omus closed this Jan 4, 2024
@omus
Copy link
Contributor Author

omus commented Jan 4, 2024

I think the right alternate approach is to have this action specify all of the registries you want to add. That way you can opt-in and out of using General. Doing that however probably also leads to addressing #20 as well.

@omus omus deleted the cv/skip-general-registry-clone branch January 4, 2024 21:07
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.

1 participant