Skip to content

third_part peewee #4262

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

Closed
wants to merge 37 commits into from
Closed

third_part peewee #4262

wants to merge 37 commits into from

Conversation

trim21
Copy link
Contributor

@trim21 trim21 commented Jun 25, 2020

  • EXCLUDED
  • SQL
  • JOIN
  • DJANGO_MAP
  • OP
  • DQ
  • chunked
  • ImproperlyConfigured
  • Field
  • UUIDField
  • BinaryUUIDField
  • IPField
  • BlobField
  • DoubleField
  • TimeField
  • DateTimeField
  • BareField
  • DateField
  • DecimalField
  • BooleanField
  • AutoField
  • BigAutoField
  • TimestampField
  • CharField
  • FixedCharField
  • IntegerField
  • BigIntegerField
  • SmallIntegerField
  • FloatField
  • BitField
  • BigBitField
  • TextField
  • ForeignKeyField
  • PrimaryKeyField
  • ManyToManyField
  • IdentityField
  • Table
  • Tuple
  • Case
  • Database
  • MySQLDatabase
  • SqliteDatabase
  • PostgresqlDatabase
  • Value
  • prefetch
  • Check
  • Model
  • DeferredThroughModel
  • fn
  • Column
  • SchemaManager
  • DataError
  • NotSupportedError
  • InterfaceError
  • DatabaseError
  • ProgrammingError
  • OperationalError
  • InternalError
  • IntegrityError
  • AsIs
  • QualifiedNames
  • Select
  • Cast
  • ValuesList
  • DoesNotExist
  • Context
  • Window
  • Index
  • ModelIndex
  • CompositeKey
  • DeferredForeignKey
  • Proxy
  • DatabaseProxy

@srittau
Copy link
Collaborator

srittau commented Jun 26, 2020

Just two things I noticed while skimming through the stubs:

  • Just use foo: Any for fields, instead of foo: Any = .... The assignment is redundant.
  • When you have not typed an argument, just leave out the type, instead of using Any. For example foo(x, y) -> None: ..., instead of foo(x: Any, y: Any) -> None: .... This makes it clear that the argument is not typed yet, but practically it is identical to Any.

@trim21
Copy link
Contributor Author

trim21 commented Jun 26, 2020

Thanks for your suggestions

@trim21
Copy link
Contributor Author

trim21 commented Jun 26, 2020

Most of code are generated by mypy stubgen, I'm still working on it.

@vmotso
Copy link

vmotso commented Aug 5, 2020

Hi there

My company is very interested in this PR.
@trim21 can you tell us approximately when this PR is planned to be completed?
We will be very grateful.

Thanks

@trim21
Copy link
Contributor Author

trim21 commented Aug 5, 2020 via email

@gvanrossum
Copy link
Member

May I suggest that it’s probably better to publish this as a separate package following PEP 561.

@jtschoonhoven jtschoonhoven mentioned this pull request Nov 30, 2020
72 tasks
@srittau
Copy link
Collaborator

srittau commented Jan 23, 2021

Just a heads up: Next week we will reshuffle the layout of typeshed. This means that the files in this PR will also need to be moved. This shouldn't be much of a problem, though, as there should be no conflicts.

@trim21 trim21 closed this Jan 23, 2021
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.

4 participants