olirice / alembic_utils

An alembic/sqlalchemy extension for migrating sql views, functions, triggers, and policies
MIT License
193 stars 42 forks source link

Fix docstring for replaceable_entity.register_entities #98

Closed pacanada closed 1 year ago

pacanada commented 1 year ago

Motivation: I was trying to include a postgres view in the alembic context. When using alembic_utils.replaceable_entity.register_entities I realised that actually the description for the entity_types argument is not accurate: instead of List[str] should be List[Type[ReplaceableEntity]].

Example of usage: https://github.com/olirice/alembic_utils/blob/master/src/test/test_depends.py#:~:text=entity_types%3D%5B-,PGView,-%5D)

Changes:

olirice commented 1 year ago

thank you