Skip to content

Port decimal stub to Python 2#95

Merged
gvanrossum merged 4 commits into
python:masterfrom
youtux:feature/decimal-2.7
Feb 26, 2016
Merged

Port decimal stub to Python 2#95
gvanrossum merged 4 commits into
python:masterfrom
youtux:feature/decimal-2.7

Conversation

@youtux

@youtux youtux commented Feb 26, 2016

Copy link
Copy Markdown
Contributor

Python 2/3 specific code has been commented and TODOs were added, according to #94 (comment).

@gvanrossum

Copy link
Copy Markdown
Member

I apologize for the advice I gave earlier. I think it's better to have separate decimal.pyi files for 2.7 and 3 after all. The number of sections with version-specific comments in this version scares me.

@youtux

youtux commented Feb 26, 2016

Copy link
Copy Markdown
Contributor Author

Yeah, the single-file strategy would make sense if mypy supported version-specific comments, but until that moment it's better to split.

gvanrossum added a commit that referenced this pull request Feb 26, 2016
@gvanrossum gvanrossum merged commit 0a49691 into python:master Feb 26, 2016
@youtux youtux deleted the feature/decimal-2.7 branch February 26, 2016 21:21
gvanrossum pushed a commit that referenced this pull request Mar 8, 2016
This broke our build.

This reverts commit 0a49691, reversing
changes made to c8630a2.
@gvanrossum

Copy link
Copy Markdown
Member

I'm really sorry, I had to roll this back because there were some deficiencies. For example:

  • __div__ and __rdiv__ were missing
  • comparison to float should be possible

@youtux youtux restored the feature/decimal-2.7 branch March 8, 2016 19:23
@youtux

youtux commented Mar 8, 2016

Copy link
Copy Markdown
Contributor Author

I'll try to fix it as soon as I can. Thank you for pointing that out.

hswong3i pushed a commit to alvistack/python-typeshed that referenced this pull request May 25, 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