Skip to content

Conversation

@Secrus
Copy link
Collaborator

@Secrus Secrus commented Aug 5, 2022

Pull Request Check List

  • Added tests for changed code. -> No new code was added
  • Updated documentation for changed code. -> Code was updated, not changed

Changes

  • Added flake8 plugins and applied changes required by those

    • rewrite of Yoda comparisons
    • simplification of comprehensions and conditionals
    • rewrite of string formatting to f-strings
    • change imports from relative to absolute
  • added from __future__ import annotations on most files

  • migrated code to Python 3.7 syntax (pyupgrade)

  • improved existing type annotations

  • added mypy type checker and exclusion list (for incremental adoption of typing)

  • updated dependencies

  • small cleanup of linter configs

@sdispater sdispater merged commit 8ef497a into python-pendulum:master Aug 6, 2022
@sdispater
Copy link
Collaborator

Thanks a lot!

@Secrus Secrus deleted the refresh branch August 6, 2022 11:13
@Secrus Secrus mentioned this pull request Aug 7, 2022
2 tasks
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.

2 participants