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

Google sync #724

Merged
merged 12 commits into from
Nov 3, 2020
Merged

Google sync #724

merged 12 commits into from
Nov 3, 2020

Conversation

rchen152
Copy link
Contributor

@rchen152 rchen152 commented Nov 3, 2020

No description provided.

martindemello and others added 12 commits November 2, 2020 18:22
If the module name is "__builtin__", add {True, False, None, __debug__}
manually to the constant list after parsing the pyi file.

Also adds some minor fixes:
* convert NamedType("nothing") to NothingType()
* fix some noncompliant `def`s

PiperOrigin-RevId: 338530575
PiperOrigin-RevId: 338742906
All of pytd/**/*.pytd is parsing now except for the shadowed module in
attr/__init__.pytd

PiperOrigin-RevId: 338747733
PiperOrigin-RevId: 339551003
Reachability metrics was originally recorded as CacheMetrics. However, the reachability mechanism is not actually a cache, but a dynamically-grown adjacency matrix. The idea of cache hits and misses is not relevant here.

PiperOrigin-RevId: 339931062
We need to be able to round-trip pyi files with the new parser.

PiperOrigin-RevId: 340283188
… place.

We ultimately want to replace the `~` with a legal python character.

PiperOrigin-RevId: 340286094
PiperOrigin-RevId: 340330922
The extra newline displeases oss pylint:
https://travis-ci.org/github/google/pytype/jobs/741022897.

PiperOrigin-RevId: 340360158
@rchen152 rchen152 merged commit 7755e3c into master Nov 3, 2020
@google-cla google-cla bot added the cla: yes label Nov 3, 2020
@rchen152 rchen152 deleted the google_sync branch November 3, 2020 03:26
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.

3 participants