Skip to content

Remove py2/py3 commentary in the dev build docs - #8289

Merged
bsipocz merged 2 commits into
astropy:masterfrom
parejkoj:remove-py2-develop-doc
Dec 31, 2018
Merged

bsipocz merged 2 commits into
astropy:masterfrom
parejkoj:remove-py2-develop-doc

Conversation

@parejkoj

Copy link
Copy Markdown
Contributor

This section was very confusing: we should just use develop, and there is no such thing as python2 anymore!

@bsipocz approved this change in person! ;-)

@codecov

codecov Bot commented Dec 15, 2018 •

Copy link
Copy Markdown

Codecov Report

Merging #8289 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #8289   +/-   ##
=======================================
  Coverage   86.92%   86.92%           
=======================================
  Files         384      384           
  Lines       57995    57995           
  Branches     1060     1060           
=======================================
  Hits        50413    50413           
  Misses       6967     6967           
  Partials      615      615

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 52d1c24...9308f33. Read the comment docs.

If you are using python 3 you will need to re-install after making changes to
the Astropy source code. Re-installing goes much faster than the initial install
because it typically does not require new compilation.
python setup.py develop

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Or better:

pip install -e .

@pllim

pllim commented Dec 17, 2018

Copy link
Copy Markdown
Member

There was a long discussion back in #7295 regarding using python vs python3, causing the bot to close it after a long time without resolution.

@bsipocz

bsipocz commented Dec 17, 2018

Copy link
Copy Markdown
Member

@pllim - I still stand by this PR. Almost a year passed on since the previous one, and python is even more Python 3, and I feel python3 would cause even more confusion.

@bsipocz
bsipocz merged commit add7e80 into astropy:master Dec 31, 2018
@bsipocz

bsipocz commented Dec 31, 2018

Copy link
Copy Markdown
Member

Thanks @parejkoj!

bsipocz added a commit that referenced this pull request Dec 31, 2018
Remove py2/py3 commentary in the dev build docs
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.

4 participants