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

Add julia keyword argument to deploydocs #58

Merged
merged 2 commits into from
Jun 21, 2018
Merged

Add julia keyword argument to deploydocs #58

merged 2 commits into from
Jun 21, 2018

Conversation

fredrikekre
Copy link
Contributor

Hello there!
This is an automated pull request submitted by @fredrikekre to update your call do Documenter.deploydocs. The julia keyword argument to Documenter.deploydocs will be required in the future instead of defaulting to julia = "nightly" (see JuliaDocs/Documenter.jl#712, JuliaDocs/Documenter.jl#526). Please take a look at this PR and ping @fredrikekre if you have any questions.

will be required in the future instead of automatically
defaulting to `julia = "nightly"
(see JuliaDocs/Documenter.jl#712,
JuliaDocs/Documenter.jl#526).

This commit adds `julia = "nightly"` to `Documenter.deploydocs`.
@codecov-io
Copy link

codecov-io commented Apr 27, 2018

Codecov Report

Merging #58 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master      #58   +/-   ##
=======================================
  Coverage   93.85%   93.85%           
=======================================
  Files           6        6           
  Lines         358      358           
=======================================
  Hits          336      336           
  Misses         22       22

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 3689d9f...25b8c8e. Read the comment docs.

@nalimilan
Copy link
Member

@fredrikekre Thanks for the automated PRs! Though wouldn't "0.6" be a better choice in general, given that "nightly" frequently breaks? That would be the occasion to fix packages.

@fredrikekre
Copy link
Contributor Author

I agree that that is better in general, and it was only ~40 or so registered packages that didn't specify this kwarg at all, most others specify 0.6. I opted for "nightly" since that was the previous default.

@kleinschmidt
Copy link
Member

okay to merge this?

@nalimilan nalimilan merged commit 73c21ac into JuliaStats:master Jun 21, 2018
@fredrikekre fredrikekre deleted the deploydocs branch June 21, 2018 13:35
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