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

Switch to Project.toml, cap Documenter #110

Merged
merged 3 commits into from
Oct 21, 2018

Conversation

mortenpi
Copy link
Contributor

Use a Project.toml, instead of a mix of a REQUIRE file + Pkg.add commands to define the environment for the doc build. Also, cap Documenter to 0.19 due to upcoming breaking changes.

I am basically just playing around here a bit, trying to figure out what would be a good pattern to use for repositories like this that are not really packages. Doing it this way, I don't think you can have PlotDocs as a package in an environment, so I am guessing it might break people's workflows?

Use a Project.toml, instead of a mix of REQUIRE + Pkg.add commands to
define the environment for the doc build.

Also, cap Documenter to 0.19 due to upcoming breaking changes

  https://discourse.julialang.org/t/psa-documenter-jl-breaking-changes-version-capping/16431
Should fix the Travis failure.
@daschw
Copy link
Member

daschw commented Oct 21, 2018

Thanks @mortenpi!I don't think PlotDocs is used anywhere else than in the make.jl here. It was not even working until recently, so I don't think this will break anyone's workflow.

@daschw daschw merged commit 0f63194 into JuliaPlots:master Oct 21, 2018
@mortenpi mortenpi deleted the update-documenter-build branch October 21, 2018 22:45
@mortenpi mortenpi mentioned this pull request Oct 21, 2018
@mkborregaard mkborregaard mentioned this pull request Oct 22, 2018
BeastyBlacksmith pushed a commit that referenced this pull request Jun 7, 2022
Switch to Project.toml, cap Documenter
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.

None yet

2 participants