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

[EXP] Undo sphinx-gallery hack #3

Merged
merged 1 commit into from Jun 22, 2017

Conversation

pllim
Copy link

@pllim pllim commented Jun 22, 2017

Do not merge

This is to see if the Travis hack is really necessary for doc build to pass. The test should kick off on my fork.

UPDATE: Oh, you have Travis set up here too. Let's just look at this one then.

@bsipocz
Copy link
Owner

bsipocz commented Jun 22, 2017

This passed! So I suggest to remove the hack for now, but keep an eye on the sphinx-gallery releases, and whenever your fix gets in a release, revise the version number requirement in the matrix.

@bsipocz bsipocz merged commit 1f9f1f5 into bsipocz:astropy_helpers_v2.0dev Jun 22, 2017
@pllim pllim deleted the pr6140-no-hack branch June 22, 2017 18:36
@pllim
Copy link
Author

pllim commented Jun 22, 2017

Okay, I opened astropy#6266 so we don't forget about it. Thanks!

bsipocz pushed a commit that referenced this pull request May 16, 2018
Don't serialize format as it is now managed with the fits code
bsipocz pushed a commit that referenced this pull request Oct 24, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants