Skip to content

Update WCS docs to use the new layout. - #267

Merged
eteq merged 1 commit into
astropy:masterfrom
mdboom:wcs/docs-update
Jun 11, 2012
Merged

eteq merged 1 commit into
astropy:masterfrom
mdboom:wcs/docs-update

Conversation

@mdboom

@mdboom mdboom commented Jun 8, 2012

Copy link
Copy Markdown
Contributor

This reorganizes the WCS docs to use the new layout.

The properties and attributes have the same problem that methods had prior to the fix in #217: they only have a summary line in the table -- they do not have a link to the full docstring (which is missing).

This is also affected by #231 (which isn't closed yet -- I haven't tried merging that with this to see if it helps, but I suspect it does).

@eteq

eteq commented Jun 8, 2012

Copy link
Copy Markdown
Member

I merged this with master, #268, and #231, and the result is at http://eteq.github.com/astropy/ . It seems mostly fine, although I notice the summary tables for the classes seem to still have the first-line-cutting-off problem. I would suggest we get all this merged first before addressing that, though, because it gets difficult to manage fixing something that has parts in 3 different PRs...

@mdboom

mdboom commented Jun 11, 2012

Copy link
Copy Markdown
Contributor Author

Sure. Why don't I merge this in advance of resolving the infrastructural details. It's always nice to test the doc building by having as much content as possible. any objections?

@eteq

eteq commented Jun 11, 2012

Copy link
Copy Markdown
Member

Agreed - I'll go ahead and merge this now, and then I can use it to try to fix the classes problem in #231.

eteq added a commit that referenced this pull request Jun 11, 2012
Update WCS docs to use the new layout.
@eteq
eteq merged commit 9a18f59 into astropy:master Jun 11, 2012
keflavich pushed a commit to keflavich/astropy that referenced this pull request Oct 9, 2013
Update WCS docs to use the new layout.
@mdboom
mdboom deleted the wcs/docs-update branch May 21, 2014 23:59
astrofrog pushed a commit to astrofrog/astropy that referenced this pull request Jun 12, 2019
jeffjennings pushed a commit to jeffjennings/astropy that referenced this pull request Jul 2, 2025
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