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

bpo-28556: Various updates to typing #28

Merged
merged 3 commits into from
Feb 13, 2017

Conversation

ilevkivskyi
Copy link
Member

This PR contains various updates from upstream python/typing repo:

  • Added typing.Counter and typing.ChainMap generics
  • More flexible typing.NamedTuple
  • Improved generic ABC caching
  • More tests
  • Bugfixes
  • Other updates

This should be backported to 3.6 and 3.5 branches.

@gvanrossum
Copy link
Member

Oh, this should probably also include a Misc/NEWS update.

@ilevkivskyi
Copy link
Member Author

I added a short entry (note that there is no issue for this, so that I referenced this PR #28)

@gvanrossum
Copy link
Member

gvanrossum commented Feb 12, 2017 via email

@ilevkivskyi
Copy link
Member Author

OK, I added the reference to the omnibus issue.

@codecov
Copy link

codecov bot commented Feb 12, 2017

Codecov Report

Merging #28 into master will increase coverage by 0.01%.
The diff coverage is 94.64%.

@@            Coverage Diff             @@
##           master      #28      +/-   ##
==========================================
+ Coverage   82.37%   82.38%   +0.01%     
==========================================
  Files        1427     1428       +1     
  Lines      350948   351138     +190     
==========================================
+ Hits       289088   289285     +197     
+ Misses      61860    61853       -7

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 e7ffb99...400be95. Read the comment docs.

@ilevkivskyi ilevkivskyi changed the title Various updates to typing bpo-28556: Various updates to typing Feb 12, 2017
@gvanrossum
Copy link
Member

I'm hoping some core dev will take kindly to this and merge it.

@brettcannon
Copy link
Member

LGTM, but I don't have the time to do the merge + cherry-picks on this.

@Mariatta Mariatta merged commit b692dc8 into python:master Feb 13, 2017
@Mariatta
Copy link
Member

Will cherry-pick shortly

Mariatta referenced this pull request in Mariatta/cpython Feb 13, 2017
 various updates from upstream python/typing repo:

- Added typing.Counter and typing.ChainMap generics
- More flexible typing.NamedTuple
- Improved generic ABC caching
- More tests
- Bugfixes
- Other updates

* Add Misc/NEWS entry

* Add issue number

(cherry picked from commit b692dc8)
Mariatta referenced this pull request in Mariatta/cpython Feb 13, 2017
 various updates from upstream python/typing repo:

- Added typing.Counter and typing.ChainMap generics
- More flexible typing.NamedTuple
- Improved generic ABC caching
- More tests
- Bugfixes
- Other updates

* Add Misc/NEWS entry

* Add issue number

(cherry picked from commit b692dc8)
@Mariatta
Copy link
Member

Cherry-picked in:
#78
#77

Mariatta added a commit that referenced this pull request Feb 13, 2017
various updates from upstream python/typing repo:

- Added typing.Counter and typing.ChainMap generics
- More flexible typing.NamedTuple
- Improved generic ABC caching
- More tests
- Bugfixes
- Other updates

* Add Misc/NEWS entry

* Add issue number

Contributed by Ivan Levkivskyi @ilevkivskyi

(cherry picked from commit b692dc8)
@ilevkivskyi ilevkivskyi deleted the update-typing branch February 13, 2017 22:33
@ilevkivskyi
Copy link
Member Author

@Mariatta Thank you very much for taking care of this!

@gvanrossum
Copy link
Member

gvanrossum commented Feb 13, 2017 via email

Mariatta added a commit that referenced this pull request Feb 14, 2017
various updates from upstream python/typing repo:

- Added typing.Counter and typing.ChainMap generics
- More flexible typing.NamedTuple
- Improved generic ABC caching
- More tests
- Bugfixes
- Other updates
* Add Misc/NEWS entry


(cherry picked from commit b692dc8)
jaraco pushed a commit that referenced this pull request Dec 2, 2022
- Ensure travis job has kicked off before checking for `pending` status.
jaraco pushed a commit to jaraco/cpython that referenced this pull request Feb 17, 2023
jaraco added a commit to jaraco/cpython that referenced this pull request Feb 17, 2023
* Use `extend-ignore` in flake8 config

This option allows to add extra ignored rules to the default list
instead of replacing it.

The default exclusions are: E121, E123, E126, E226, E24, E704,
W503 and W504.

Fixes python#28.

Refs:
* https://github.com/pypa/setuptools/pull/2486/files#r541943356
* https://flake8.pycqa.org/en/latest/user/options.html#cmdoption-flake8-extend-ignore
*
https://flake8.pycqa.org/en/latest/user/options.html#cmdoption-flake8-ignore

* Enable complexity limit. Fixes jaraco/skeleton#34.

* Replace pep517.build with build (python#37)

* Replace pep517.build with build

Resolves python#30

* Prefer simple usage

Co-authored-by: Jason R. Coombs <jaraco@jaraco.com>

* Use license_files instead of license_file in meta (python#35)

Singular `license_file` is deprecated since wheel v0.32.0.

Refs:
* https://wheel.readthedocs.io/en/stable/news.html
* https://wheel.readthedocs.io/en/stable/user_guide.html#including-license-files-in-the-generated-wheel-file

Co-authored-by: Jason R. Coombs <jaraco@jaraco.com>
nanjekyejoannah added a commit to nanjekyejoannah/cpython that referenced this pull request May 22, 2023
28: test clean final r=ltratt a=nanjekyejoannah

This PR does the final cleanups to address the following:
- Allow higher recursion limit for some tests
- skip some tests that require very high recursion limits, I used the allowed limit by convention for the first item above
- call reset to allow warning propagation in some tests

Note:

Existing py3k tests fail with a segfault with -3 flag if not run individually, deleting this context manager in over 300 files will take forever, lets only run `test_py3kwarn` and the rest should be tested locally if modified. 

It took me long to know the source of this segfault, I have solved any issues with new context manager I introduce.

If this PR is merged, we can modify the CI, I am going back to implementing the warnings now.

Co-authored-by: Joannah Nanjekye <jnanjeky@unb.ca>
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.

5 participants