RFR: Get agreement before PR
Markus KARG
duke at openjdk.org
Sun May 11 09:39:02 UTC 2025
On Mon, 5 May 2025 19:05:41 GMT, Jesper Wilhelmsson <jwilhelm at openjdk.org> wrote:
> Added a note on getting proper agreement on a change before creating a PR for it.
src/guide/contributing-to-an-open-jdk-project.md line 65:
> 63: ### 3. Find a Sponsor
> 64:
> 65: Socializing your change on the mailing lists also prevents the surprise that would otherwise make the community choke on their morning coffee when they see a huge patch in a new, unknown PR. As a new developer in the community you'll need to make a few friends that agree with your change. There are many good reasons to make friends, but the one relevant here is that for your first changes you'll need a [Sponsor](https://openjdk.org/bylaws#sponsor) to facilitate the integration of your work. The [Sponsor](https://openjdk.org/bylaws#sponsor) will perform any number of administrative tasks like JBS updates, additional testing, etc. It's usual for a [Sponsor](https://openjdk.org/bylaws#sponsor) to also be a reviewer of a change and thus familiar with it, but it's not a requirement.
Imho "for your first changes" is misleading here. I contributed lots of commits, but still do need a sponsor even after years. We should find a wording which does not give the impression that contributing many commits over several years will change anything on the fact that a sponsor *still* is mandatory for each PR.
-------------
PR Review Comment: https://git.openjdk.org/guide/pull/150#discussion_r2083467228
More information about the guide-dev
mailing list