Xymon Mailing List Archive search

Review rules, the backlog, and a proposal for when nobody comes

list Roland Rosenfeld
Mon, 14 Sep 2026 18:10:31 +0200
Message-Id: <user-653dea215c4b@xymon.invalid>

Hi Bruno!

On Mo, 14 Sep 2026, Bruno Manzoni via Xymon wrote:
To resolve this I propose to add a delay before bypassing a PR:

- documentation, tests — 1 week (current 0 days)
- build and CI — 2 weeks (current 0 days)
- bug fixes — 1 month (current: never)
- everything else — 2 months (current: never)
I object this proposal.  I can live with AI generated code (don't like
it, but accept it), but at least one human should review the changes.
Maybe it's okay, if AI generates the change and you personally (not
the AI, you are using) review the generated change.

I know, that I should review #219, but I'm currently on vacation, so I
have not much time nor a big display nor a test environment, so I
cannot review this big job.  The review steps the AI did itself
(finding some issues) shows that this isn't a trivial PR and reviewing
it is important.  I think that the review should include a end-to-end
test on a real life system to see, whether this works as expected.

BTW: From my point of view, we should release early/often and the
current snapshot is much better than 4.3.30 ever was.  So why not just
release 4.3.31 as is and continue to work on 4.3.32?  If your AI stays
as active as it currently is, we will never reach a state where no PRs
for 4.3.31 are ready for review, since the AI generates them much
faster than any human can review them.

Greetings
Roland