-
Notifications
You must be signed in to change notification settings - Fork 88
Open
Labels
Description
TODO we should consolidate these forward refs with their definitions in config.py workflow_service uses this, which is why it's here instead
AutoPR/autopr/models/executable.py
Lines 93 to 104 in 1d818f4
| if value in cls.reserved_keywords: | |
| raise ValueError(f"{value} is a reserved keyword") | |
| return str.__new__(cls, value) | |
| # TODO we should consolidate these forward refs with their definitions in `config.py` | |
| # workflow_service uses this, which is why it's here instead | |
| ExecutableForwardRef = Union[ | |
| ExecutableId, | |
| # ControlWords, | |
| ForwardRef("ActionConfigs"), # pyright: ignore[reportGeneralTypeIssues] | |
| ForwardRef("WorkflowInvocation"), # pyright: ignore[reportGeneralTypeIssues] |
When resolving this TODO, you can start by identifying all the forward references in your code and their corresponding definitions in config.py. Then, consolidate these forward references with their definitions in config.py to eliminate any duplication. Finally, make sure to update any references to the consolidated definitions in your code.