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

don't use PkgServers in Documenter job #296

Closed

Conversation

simeonschaub
Copy link
Collaborator

The version of the registry served by PkgServers can sometimes lag a
little bit behind, leading to issues such as in JuliaDiff/FiniteDifferences.jl#173.
The julia-buildpkg action actually already does this here:
https://github.com/julia-actions/julia-buildpkg/blob/1b0a69ab74709d3707f86249fce805049212961a/action.yml#L22

The version of the registry served by PkgServers can sometimes lag a
little bit behind, leading to issues such as in JuliaDiff/FiniteDifferences.jl#173.
The `julia-buildpkg` action actually already does this here:
https://github.com/julia-actions/julia-buildpkg/blob/1b0a69ab74709d3707f86249fce805049212961a/action.yml#L22
@DilumAluthge
Copy link
Member

The better approach is not to completely opt-out of the Pkg server, but rather to use Git for the General registry. Then you can still use the Pkg server by default, and call back to GitHub as necessary.

So I would recommend that instead of just opting out of the Pkg server, take the approach from e.g. julia-actions/julia-runtest#17.

@codecov
Copy link

codecov bot commented Jun 9, 2021

Codecov Report

Merging #296 (b33e580) into master (4302ecb) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #296   +/-   ##
=======================================
  Coverage   94.40%   94.40%           
=======================================
  Files          20       20           
  Lines         608      608           
=======================================
  Hits          574      574           
  Misses         34       34           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 4302ecb...b33e580. Read the comment docs.

@simeonschaub
Copy link
Collaborator Author

Bump. @DilumAluthge does that look good to you?

@oxinabox
Copy link
Collaborator

oxinabox commented Sep 2, 2021

resolved by
#304

@oxinabox oxinabox closed this Sep 2, 2021
@simeonschaub simeonschaub deleted the sds/docs_no_pkg_server branch September 2, 2021 13:45
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.

3 participants