Fix typing of partial_reordering

[Flaab123]

The typing of the partial_reordering argument of batch_alter_table is not correct.

Description

As stated in the documentation and docstring, the batch_alter_table() function expects a list of tuples for the partial_reordering argument. The current typing suggests the user to insert a tuple containing Any into the argument, which doesn't align with the docstring.

Checklist

This pull request is:

• [ x] A documentation / typographical error fix
  • Good to go, no issue or tests are needed
• A short code fix
  • please include the issue number, and create an issue if none exists, which
    must include a complete example of the issue. one line code fixes without an
    issue and demonstration will not be accepted.
  • Please include: Fixes: #<issue number> in the commit message
  • please include tests. one line code fixes without tests will not be accepted.
• A new feature implementation
  • please include the issue number, and create an issue if none exists, which must
    include a complete example of how the feature would look.
  • Please include: Fixes: #<issue number> in the commit message
  • please include tests.

Have a nice day!

[sqla-tester]

OK, this is sqla-tester setting up my work on behalf of zzzeek to try to get revision afd9a70 of this pull request into gerrit so we can run tests and reviews and stuff

[sqla-tester]

New Gerrit review created for change afd9a70: https://gerrit.sqlalchemy.org/c/sqlalchemy/alembic/+/6521

[zzzeek]

I think these Any are strings, we can use newer style annotations now also

[sqla-tester]

OK, this is sqla-tester setting up my work on behalf of zzzeek to try to get revision f0a0c1a of this pull request into gerrit so we can run tests and reviews and stuff

[sqla-tester]

Patchset f0a0c1a added to existing Gerrit review https://gerrit.sqlalchemy.org/c/sqlalchemy/alembic/+/6521

[sqla-tester]

Michael Bayer (zzzeek) wrote:

our tooling does not support the | None thing yet so i had to change it back to Optional[]. thanks for the patch

View this in Gerrit at https://gerrit.sqlalchemy.org/c/sqlalchemy/alembic/+/6521

[sqla-tester]

Gerrit review https://gerrit.sqlalchemy.org/c/sqlalchemy/alembic/+/6521 has been merged. Congratulations! :)