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

Conversation

Projects
None yet
3 participants
@astrofrog
Member

astrofrog commented Feb 17, 2015

@eteq

This comment has been minimized.

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

This comment has been minimized.

Member

astrofrog commented Feb 18, 2015

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

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

@eteq

This comment has been minimized.

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

This comment has been minimized.

Member

Cadair commented Feb 18, 2015

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

@astrofrog

This comment has been minimized.

Member

astrofrog commented Feb 18, 2015

@Cadair - ok, fixed

@eteq

This comment has been minimized.

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

Merge pull request #3513 from astrofrog/known-issues
Added a couple of known issues

@eteq eteq merged commit d55c2d0 into astropy:master Feb 18, 2015

3 checks passed

continuous-integration/appveyor AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
coverage/coveralls First build on master at 75.91%
Details

eteq added a commit that referenced this pull request Feb 18, 2015

Merge pull request #3513 from astrofrog/known-issues
Added a couple of known issues
@eteq

This comment has been minimized.

Member

eteq commented Feb 18, 2015

Backported to v1.0.x in fd33850

@astrofrog astrofrog deleted the astrofrog:known-issues branch Jul 5, 2016

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