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

Docs aren't getting deployed? #1241

Closed
tlnagy opened this issue Jan 8, 2019 · 10 comments
Closed

Docs aren't getting deployed? #1241

tlnagy opened this issue Jan 8, 2019 · 10 comments

Comments

@tlnagy
Copy link
Member

tlnagy commented Jan 8, 2019

@bjarthur any ideas on why Travis is skipping doc deployment? https://travis-ci.org/GiovineItalia/Gadfly.jl/jobs/476704204

Documenter: populating indices.
Documenter: rendering document.
Documenter: skipping docs deployment.
@bjarthur
Copy link
Member

bjarthur commented Jan 8, 2019

not sure why it didn't work this time, but it did last time: https://travis-ci.org/GiovineItalia/Gadfly.jl/builds/475180576?utm_source=github_status&utm_medium=notification

@tlnagy
Copy link
Member Author

tlnagy commented Jan 8, 2019

It looks like it skips in your link too:

Documenter: setting up build directory.
Documenter: expanding markdown templates.
Documenter: building cross-references.
Documenter: running document checks.
 > checking for missing docstrings.
 > running doctests.
 > checking footnote links.
Documenter: populating indices.
Documenter: rendering document.
Documenter: skipping docs deployment.
Documenter: You can set DOCUMENTER_DEBUG to "true" in Travis to see more information.

@bjarthur
Copy link
Member

bjarthur commented Jan 8, 2019

@tlnagy
Copy link
Member Author

tlnagy commented Jan 8, 2019

http://gadflyjl.org/dev/ is out of date as well because there are some bugs in the plots that have been fixed on master, e.g. opacity in the Geom.ribbon is broken in the docs, but the exact same code works on master. This was fixed at some point after 1.0.1, I think in #1231

@tlnagy
Copy link
Member Author

tlnagy commented Jan 8, 2019

Line L34 in .travis.yml refers to after the docs have been deployed. Deployment should not be skipped. You can verify that at your first link:

Documenter: skipping docs deployment.
Documenter: You can set DOCUMENTER_DEBUG to "true" in Travis to see more information.
...
Skipping the after_success step, as specified in the configuration.

@bjarthur
Copy link
Member

bjarthur commented Jan 8, 2019

cc @mortenpi can you please help with a travis doc deployment question?

@tlnagy
Copy link
Member Author

tlnagy commented Jan 8, 2019

I've started up the VM with the DOCUMENTER_DEBUG key set to true. Hopefully that'll give us a better idea of what is going on: https://travis-ci.org/GiovineItalia/Gadfly.jl/jobs/476704204

@mortenpi
Copy link
Contributor

mortenpi commented Jan 8, 2019

The build linked in the OP looks like a PR build for #1240, right? The PR builds are not supposed to deploy docs (among other reasons, they don't have access to DOCUMENTER_KEY).

The last build for the master branch deployed fine as far as I can tell: https://travis-ci.org/GiovineItalia/Gadfly.jl/builds/475701320

@tlnagy
Copy link
Member Author

tlnagy commented Jan 8, 2019

Ah, good catch. My bad. I wonder why the plots are broken in the documentation, but build fine locally on master. That's a separate problem and I'll open a separate issue for that.

@tlnagy tlnagy closed this as completed Jan 8, 2019
@mortenpi
Copy link
Contributor

mortenpi commented Jan 8, 2019

Regarding plots, I didn't look into it, but could it be something similar to JuliaDocs/Documenter.jl#921?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants