Adam Majer

adamm created branch pending-reviews-not-seen in git-workflow/autogits

2026-02-26 17:11:26 +01:00

adamm pushed to pending-reviews-not-seen at git-workflow/autogits

2026-02-26 17:11:26 +01:00

3ae7cf0d82 tests

ff621a6d70 common: check for old pending request reviews

Compare 2 commits »

adamm pushed to main at git-workflow/autogits

2026-02-26 13:16:07 +01:00

5669083388 pr: handle case of nil user in reviews

adamm commented on issue git-workflow/autogits#139

2026-02-26 12:22:22 +01:00

bots in containers leave many defunct processes

I would be interested to see the patch regardless. I saw this issue running in container too.

adamm commented on pull request git-workflow/autogits#138

2026-02-25 18:02:44 +01:00

WIP: fix: reviews may be ahead of timeline

ok, I think I understand where the problem is. Let me make a unit test and fix and you can merge that into this pr?

adamm deleted branch new-packages from git-workflow/autogits

2026-02-25 17:53:57 +01:00

adamm commented on pull request git-workflow/autogits#138

2026-02-25 17:32:54 +01:00

WIP: fix: reviews may be ahead of timeline

Basically, you should end up in same situation. A race condition, ie. you add a review trigger that is not seen in timeline, means that the timeline is modified asynchronously after requests…

adamm commented on pull request git-workflow/autogits#138

2026-02-25 17:24:34 +01:00

WIP: fix: reviews may be ahead of timeline

I think this is correct. The timeline could be behind rawReviews, but those rawReviews that are not on the timeline should be ignored.

adamm merged pull request git-workflow/autogits#136

2026-02-25 14:49:39 +01:00

staging-updates

adamm pushed to main at git-workflow/autogits

2026-02-25 14:49:38 +01:00

582df2555b Merge branch 'staging-updates'

91d22f7eea staging: add tests for idempotency and label changes

913b8c8a4b staging: Match previous message format

e1825dc658 staging: CommentPROnce everywhere

59965e7b5c staging: comment once on PRs using timeline

Compare 18 commits »

adamm commented on pull request git-workflow/autogits#108

2026-02-25 12:53:50 +01:00

New package handling

rebased and merged.

adamm merged pull request git-workflow/autogits#108

2026-02-25 12:53:25 +01:00

New package handling

adamm pushed to new-packages at git-workflow/autogits

2026-02-25 12:53:24 +01:00

7f6bd4bc32 pr: Always allow maintainer edits in new packages

f1b807fbf6 common: replace legacy logger with standard impl

e5441bf489 test: refactor loggin in unit tests

8078ca7d4d pr: small refactor

8721aa2c14 pr: test coverage

Compare 84 commits »

adamm pushed to main at git-workflow/autogits

2026-02-25 12:52:55 +01:00

4179fb4c7b Merge branch 'new-packages'

7f6bd4bc32 pr: Always allow maintainer edits in new packages

f1b807fbf6 common: replace legacy logger with standard impl

e5441bf489 test: refactor loggin in unit tests

8078ca7d4d pr: small refactor

Compare 26 commits »

adamm pushed to main at git-workflow/autogits

2026-02-25 10:53:03 +01:00

1cf7dd79b3 Merge branch 'main' of src.opensuse.org:git-workflow/autogits

29607f922c HACK: disable close of pullrequests for now

Compare 2 commits »

adamm commented on pull request git-workflow/autogits#108

2026-02-24 23:11:00 +01:00

New package handling

Yes, this needs improvement

adamm commented on pull request git-workflow/autogits#108

2026-02-24 23:06:30 +01:00

New package handling

It's sadly used. It's a global in the application. It makes sense in the app, but it's confusing here.

adamm pushed to staging-updates at git-workflow/autogits

2026-02-24 18:23:04 +01:00

91d22f7eea staging: add tests for idempotency and label changes

adamm pushed to mergemodes at git-workflow/autogits

2026-02-24 17:51:32 +01:00

e1059e9447 pr: No need to try to merge changes

adamm pushed to staging-updates at git-workflow/autogits

2026-02-24 12:26:43 +01:00

913b8c8a4b staging: Match previous message format