Skip to content

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
MemberAuthor

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
MemberAuthor

OK, I added the reference to the omnibus issue.

@codecov
Copy link

codecovbot 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.

@ilevkivskyiilevkivskyi changed the title Various updates to typingbpo-28556: Various updates to typingFeb 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.

@MariattaMariatta merged commit b692dc8 into python:masterFeb 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)
@ilevkivskyiilevkivskyi deleted the update-typing branch February 13, 2017 22:33
@ilevkivskyi
Copy link
MemberAuthor

@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. Fixespython#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. Fixesjaraco/skeleton#34. * Replace pep517.build with build (python#37) * Replace pep517.build with build Resolvespython#30 * Prefer simple usage Co-authored-by: Jason R. Coombs <[email protected]> * 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 <[email protected]>
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 <[email protected]>
Sign up for freeto 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

@ilevkivskyi@gvanrossum@brettcannon@Mariatta@the-knights-who-say-ni