-
Notifications
You must be signed in to change notification settings - Fork 83
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.