-
Notifications
You must be signed in to change notification settings - Fork 88
Open
Labels
Description
TODO split out multiple triggered workflows into separate PRs
AutoPR/autopr/services/trigger_service.py
Lines 124 to 134 in 1d818f4
| self, | |
| event: EventUnion, | |
| ): | |
| triggers_and_contexts = self._get_triggers_and_contexts_for_event(event) | |
| # TODO split out multiple triggered workflows into separate PRs | |
| if any(trigger.verbose for trigger, _ in triggers_and_contexts): | |
| self.publish_service.verbose = True | |
| trigger_coros = await self._get_trigger_coros_for_event(triggers_and_contexts) | |
| if not trigger_coros: |
AutoPR/autopr/services/trigger_service.py
Lines 181 to 191 in 1d818f4
| await self.publish_service.close( | |
| "Closing because the PR makes no changes", | |
| ) | |
| # Else, if there are material changes | |
| elif changes_status == "modified": | |
| # TODO split out multiple triggered workflows into separate PRs | |
| automerge_triggers = [trigger for trigger in triggers if trigger.automerge] | |
| if automerge_triggers: | |
| automerge_ids = [ | |
| self._get_id_for_executable(trigger.run) for trigger in automerge_triggers | |
| ] |
To resolve this TODO, you can start by identifying which triggered workflows need to be split out. Then, create separate pull requests for each workflow, making sure to clearly document the changes made and the reasons for splitting them. Finally, review and test each pull request thoroughly before merging them into the main codebase.