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

Added a couple of known issues #3513

Merged
merged 4 commits into from Feb 18, 2015
Merged

Added a couple of known issues #3513

merged 4 commits into from Feb 18, 2015

Conversation

astrofrog
Copy link
Member

@eteq
Copy link
Member

eteq commented Feb 18, 2015

@astrofrog - this is fine, but while you're editing the known issues, can I suggest two other changes?:

  1. Re-arrange the issues so that some of the more common installation issues are at the top. Specifically, the conda/pip issue, and the locale issue. Also, perhaps move some of the issues dealing with older versions further down? (esp. the Astropy v0.2 issue, but possibly also the py 2.6.x-specific one)
  2. Can you add a table-of-contents to the top? I'm not entirely sure this will work, but a .. toctree:: directive pointing to the current file might be the way to go. I realize this appears in the sidebar in the generated docs, but I think it might still be helpful to see it inline.

@astrofrog
Copy link
Member Author

@eteq - done, and here's the current order (suggestions welcome):

screen shot 2015-02-18 at 10 02 33 am

@eteq
Copy link
Member

eteq commented Feb 18, 2015

Looks good to me except for one other minor change I'd like to suggest: "Local Errors" -> "Locale Errors in Mac OS X".

@Cadair
Copy link
Member

Cadair commented Feb 18, 2015

@eteq I have seen the locale errors on Linux as well.

@astrofrog
Copy link
Member Author

@Cadair - ok, fixed

@eteq
Copy link
Member

eteq commented Feb 18, 2015

This looks good now, so I'll go ahead and merge.

eteq added a commit that referenced this pull request Feb 18, 2015
Added a couple of known issues
@eteq eteq merged commit d55c2d0 into astropy:master Feb 18, 2015
eteq added a commit that referenced this pull request Feb 18, 2015
Added a couple of known issues
@eteq
Copy link
Member

eteq commented Feb 18, 2015

Backported to v1.0.x in fd33850

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants