Skip to content

CI/DOC Use sphinx-gallery 0.3.1 #14833

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

Conversation

lesteve
Copy link
Member

@lesteve lesteve commented Aug 28, 2019

This PR target the 0.21.X branch. I cherry-picked the commit from #14507 (merged 4 weeks ago in master).

My goal behind this is to add binder links to the stable doc (very similar to #11221 which added binder links to the dev doc). The work done in master needs sphinx-gallery 0.3.1.

I am not expecting any issues, but once the doc is built in this PR I will double-check whether I don't see any obvious problems in the generated doc.

Also what is the preferred way of changing things in a relase branch:

  • should this PR be merged (similar workflow as master)?
  • should someone push the commit directly in the 0.21.X branch (more obvious that it is a cherry-pick of a commit in master)? This seems to be the approach for bug-fixes in 0.21.X as far as I can see.

@jnothman
Copy link
Member

jnothman commented Aug 28, 2019 via email

sphinx 1.6.2 is only available in free channel (which was removed).

[doc build]
@lesteve
Copy link
Member Author

lesteve commented Aug 28, 2019

I compared the examples gallery from the stable doc and the examples gallery from the doc generated from this PR and I did not spot any difference. I also looked at a few examples and they were identical.

It seems like I will have to fix a few things related to conda removing the free channel by default though to get the CIs green again in 0.21.X ...

@lesteve
Copy link
Member Author

lesteve commented Oct 24, 2019

Closing this one, waiting for the next scikit-learn relase for the Binder links to appear in the stable doc seems a better idea.

@lesteve lesteve closed this Oct 24, 2019
@lesteve lesteve deleted the sphinx-gallery-0.3.1-for-0.21-branch branch February 9, 2021 14:36
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.

2 participants