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.

Sign up for GitHub

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

Jump to bottom

Changes to travis cause recommended travis file to fail #1228

Closed
baggepinnen opened this issue Jan 22, 2020 · 6 comments · Fixed by #1231
Closed

Changes to travis cause recommended travis file to fail #1228

baggepinnen opened this issue Jan 22, 2020 · 6 comments · Fixed by #1231
Labels
Type: Documentation

Comments

@baggepinnen
Copy link
Contributor

Previously (like a week ago?) following the instructions at
https://juliadocs.github.io/Documenter.jl/dev/man/hosting/#Travis-CI-1
and just adding the suggested stuff to the travis file worked and the documentation was built. Now, if you click the tab "View config" here
https://travis-ci.org/baggepinnen/MonteCarloMeasurements.jl/builds/640388285/config
you see that the build config validation failed. It seems travis no longer accepts a file that specifies both "matrix" and "jobs". The same travis file did work and pass the build config validation a couple of days ago though so it seems it's a change in travis' side. Changing the travis file to this solved the problem
https://travis-ci.org/baggepinnen/MonteCarloMeasurements.jl/builds/640444914/config
@fredrikekre
Copy link
Member

Yea, someone else mentioned this too, but when I looked into it (more than a month ago) I could not find anything about this in the Travis docs, and they still mention both matrix and jobs.

@fredrikekre
Copy link
Member

Can you try just collapsing them?

@baggepinnen
Copy link
Contributor Author

What does collapsing them mean?

@fredrikekre
Copy link
Member

Putting everything under matrix and jobs under matrix (or jobs since Travis claims they are synonyms?).

@baggepinnen
Copy link
Contributor Author

Yeah that's what I did to resolve the issue. Some recent threads on travis forums seem related:
https://travis-ci.community/t/custom-stage-disables-default-stage-matrix-expansion/1884/7
https://travis-ci.community/t/jobs-matrix-include-are-ignored-if-env-is-customized/6313/4

This was referenced Jan 22, 2020
Merged
Merged
bors bot added a commit to FluxML/Zygote.jl that referenced this issue Jan 24, 2020
@bors @oxinabox
Merge #479
91f6125 
479: make docs build r=MikeInnes a=oxinabox

I haven't checked if they are not building but TravisCI broke them for most who use both `jobs` and `matrix`

JuliaDocs/Documenter.jl#1228

Co-authored-by: Lyndon White <oxinabox@ucc.asn.au>
briochemc referenced this issue in jw3126/UnitfulRecipes.jl Jan 29, 2020
@briochemc
update travis
eeedb3e
@mortenpi mortenpi mentioned this issue Jan 29, 2020
Merged
@Balinus
Copy link

Balinus commented Feb 6, 2020

Had the same problem. Copy-pasted @baggepinnen .travis.yml file solved the issue. Documentation stage were not starting as expected.

@ablaom ablaom mentioned this issue Feb 6, 2020
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Type: Documentation
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Discourage use of matrix: in Travis conf JuliaDocs/Documenter.jl
4 participants
@mortenpi @Balinus @baggepinnen @fredrikekre