Skip to content

[FIX] otools-pending remove: crash when removing last pending merge#153

Merged
gurneyalex merged 1 commit intocamptocamp:masterfrom
ivantodorovich:worktree-rosy-pondering-planet
Mar 13, 2026
Merged

[FIX] otools-pending remove: crash when removing last pending merge#153
gurneyalex merged 1 commit intocamptocamp:masterfrom
ivantodorovich:worktree-rosy-pondering-planet

Conversation

@ivantodorovich
Copy link
Member

@ivantodorovich ivantodorovich commented Mar 12, 2026

When the last pending PR was removed, the merges file was deleted but the aggregator still tried to read it, causing a ConfigException.

git_aggregator.exception.ConfigException: Unable to find configuration file: /[...]/pending-merges.d/server-env.yml

Skip aggregation when the file has been removed (no merges left).

@ivantodorovich ivantodorovich force-pushed the worktree-rosy-pondering-planet branch from 57f7dff to 85ad957 Compare March 12, 2026 16:43
@ivantodorovich ivantodorovich marked this pull request as draft March 12, 2026 16:55
@ivantodorovich ivantodorovich force-pushed the worktree-rosy-pondering-planet branch 2 times, most recently from 3d88964 to 25ed07b Compare March 12, 2026 17:56
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@ivantodorovich ivantodorovich force-pushed the worktree-rosy-pondering-planet branch from 25ed07b to 3d466c9 Compare March 12, 2026 17:56
@ivantodorovich ivantodorovich marked this pull request as ready for review March 12, 2026 17:57
@gurneyalex gurneyalex merged commit 06dadda into camptocamp:master Mar 13, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants