Skip to content

Remove fractions compatibility layer since we now no longer support Python 2.6 - #4463

Merged
mhvk merged 4 commits into
astropy:masterfrom
astrofrog:remove-fractions-compat
Jan 13, 2016
Merged

mhvk merged 4 commits into
astropy:masterfrom
astrofrog:remove-fractions-compat

Conversation

@astrofrog

Copy link
Copy Markdown
Member

No description provided.

@astrofrog astrofrog added this to the v1.2.0 milestone Jan 10, 2016
@astrofrog
astrofrog force-pushed the remove-fractions-compat branch from 03aebc1 to 765f87b Compare January 10, 2016 16:42
@astrofrog
astrofrog force-pushed the remove-fractions-compat branch from 765f87b to 025ecfd Compare January 11, 2016 11:15
@astrofrog
astrofrog force-pushed the remove-fractions-compat branch from 025ecfd to f66fa84 Compare January 11, 2016 11:20
@mhvk

mhvk commented Jan 11, 2016

Copy link
Copy Markdown
Contributor

@astrofrog - this looks good, except that you missed the documentation -- see
docs/units/combining_and_defining.rst (I think OrderedDict is fine in this respect).

@mhvk mhvk self-assigned this Jan 11, 2016
@astrofrog

Copy link
Copy Markdown
Member Author

@mhvk - ah yes, good catch - thanks! (this was also the case for argparse)

@mhvk

mhvk commented Jan 13, 2016

Copy link
Copy Markdown
Contributor

Looks all OK, so merging.

mhvk added a commit that referenced this pull request Jan 13, 2016
Remove fractions compatibility layer since we now no longer support Python 2.6
@mhvk
mhvk merged commit c494d39 into astropy:master Jan 13, 2016
@embray

embray commented Jan 13, 2016

Copy link
Copy Markdown
Member

👯

@astrofrog
astrofrog deleted the remove-fractions-compat branch July 5, 2016 15:52
saimn added a commit to saimn/astropy that referenced this pull request Jun 6, 2017
Note these were only issuing a warning before importing the stdlib
modules. Follows astropy#4462 astropy#4463 astropy#4464, and similar to astropy#5975 and astropy#6157
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants