pgjdbc/pgjdbc GitHub issues and pull requests (mirror)
help / color / mirror / Atom feedFrom: sehrope (@sehrope) <[email protected]>
To: pgjdbc/pgjdbc <[email protected]>
Subject: Re: [pgjdbc/pgjdbc] PR #4075: docs: restructure site, compatibility page, changelog-driven releases
Date: Fri, 22 May 2026 12:54:19 +0000
Message-ID: <[email protected]> (raw)
In-Reply-To: <[email protected]>
References: <[email protected]>
Anything of this magnitude, whether more tooling, code change, docs change, or both (as is here) needs to have a wider acceptance before it starts. It's fine to experiment on your fork and see what comes out. It's fine to create the patch too and use it as an example of what you want to do. But it's not reasonable to make 14000+ line changes and expect others to review and sign off on them.
I really see no way of this PR ever being merged. I think we should just reject it and take it as a lesson of how __not__ to make gigantic repo wide changes.
If you want to redo the site, figure out a way to do it piecemeal that allows the incremental pieces to actually be reviewed. By humans, not just more AI. The documentation is in English and if that's not your native language, it probably means you shouldn't be wholesale replacing it as it's not possible to self review. It'd be like me replacing all the Russian message translations!
And without a meaningful self review process, you're creating an inordinate amount of work for others. Each comment or reply in this thread has generated 1000+ diffs of new items. And that's from us barely skimming through it.
view thread (64+ messages) latest in thread
reply
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Reply to all the recipients using the --to and --cc options:
reply via email
To: github://pgjdbc/pgjdbc
Cc: [email protected], [email protected]
Subject: Re: [pgjdbc/pgjdbc] PR #4075: docs: restructure site, compatibility page, changelog-driven releases
In-Reply-To: <<[email protected]>>
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
This inbox is served by agora; see mirroring instructions
for how to clone and mirror all data and code used for this inbox