Skip to content
This repository has been archived by the owner on May 25, 2022. It is now read-only.

Fixing extension warnings raised by sphinx 1.4 #224

Merged
merged 2 commits into from
Apr 15, 2016

Conversation

bsipocz
Copy link
Member

@bsipocz bsipocz commented Apr 13, 2016

There are two new warnings coming from the new 1.4 Sphinx. This causes travis failures for the sphinx builds for packages using astropy-helpers. The proposed workaround astropy/ci-helpers#78 is to set the version to a previous release, however on the long term, it's better to deal with them here.

This PR addresses the second warning, as I'm not yet sure how to fix the first one.

WARNING: while setting up extension astropy_helpers.sphinx.ext.autodoc_enhancements: directive 'autoattribute' is already registered, it will be overridden
WARNING: sphinx.ext.pngmath has been deprecated. Please use sphinx.ext.imgmath instead.

[edit: I've cancelled the appveyor build as it's irrelevant and there was a queue for other packages]

@bsipocz
Copy link
Member Author

bsipocz commented Apr 14, 2016

The first warning is coming from this https://github.com/astropy/astropy-helpers/blob/master/astropy_helpers/sphinx/ext/autodoc_enhancements.py#L75

I tried to catch the warning there and ignore it, but it didn't work. Getting the bug workaround into upstream sphinx may be the best long term solution.

bsipocz added a commit to bsipocz/astropy-helpers that referenced this pull request Apr 15, 2016
bsipocz added a commit to bsipocz/astropy-helpers that referenced this pull request Apr 15, 2016
bsipocz added a commit to bsipocz/astropy-helpers that referenced this pull request Apr 15, 2016
@eteq
Copy link
Member

eteq commented Apr 15, 2016

Well, this one looks fine so I'll go ahead and merge and create a seperate issue for the remaning one.

@eteq eteq merged commit 3a1f799 into astropy:master Apr 15, 2016
@astrofrog astrofrog modified the milestone: v1.2 Jun 2, 2016
@bsipocz
Copy link
Member Author

bsipocz commented Jun 10, 2016

@astrofrog - another one for backportation

astrofrog pushed a commit to astropy/sphinx-astropy that referenced this pull request Jan 29, 2018
…x_deprecations

Fixed one of the warnings raised by sphinx 1.4
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants