Skip to content
This repository was archived by the owner on Apr 26, 2024. It is now read-only.

Finalize 'mypy synapse/' #8363

Closed

Conversation

ShadowJonathan
Copy link
Contributor

@ShadowJonathan ShadowJonathan commented Sep 20, 2020

A cumulative PR with the final goal to be able to mypy synapse with current validation settings.

Pull Request Checklist

  • Pull request is based on the develop branch
  • Pull request includes a changelog file. The entry should:
    • Be a short description of your change which makes sense to users. "Fixed a bug that prevented receiving messages from other servers." instead of "Moved X method from EventStore to EventWorkerStore.".
    • Use markdown where necessary, mostly for code blocks.
    • End with either a period (.) or an exclamation mark (!).
    • Start with a capital letter.
  • Pull request includes a sign off
  • Code style is correct (run the linters)

Notes:

  • todo depr-35 items are placed to signal actions needing to be taken when python 3.5 is deprecated (Deprecate python 3.5 #8347)
  • type: ignore items should eventually be eliminated, or given a very good reason as to why they exist

PR To-Do list

  • Find and add all correct DictProperty type annotations in synapse.events.EventBase

Signed-off-by: Jonathan de Jong <[email protected]>

@ShadowJonathan ShadowJonathan changed the title Finalize mypy synapse/ Finalize 'mypy synapse/' Sep 20, 2020
@clokep
Copy link
Member

clokep commented Sep 21, 2020

@ShadowJonathan I'd suggest splitting this into separate PRs. It is much more likely to be reviewed and merged quickly if it is broken out!

@ShadowJonathan
Copy link
Contributor Author

I'm splitting this into some other PRs, handling a few folders at a time

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants