Feedback after first Contribution

Michael Bien mbien42 at gmail.com
Sat Oct 24 22:13:31 UTC 2020


Hello,

earlier this week I made my first contribution and wanted to leave some 
feedback from the POV of an external contributor.

First let me explain what I did to get it integrated:

Since the bugfix itself was trivial, I created a pull request right away 
(I signed the OCA before that) before discussing anything on a dev list, 
since I wanted to use the code as basis for the discussion.

The bot (which is pretty cool) quickly pointed out that the PR didn't 
have an issue assigned to it. I figured the only way to get an issue for 
me would be to either ask on the mailing list or to paste the paragraphs 
of the PR into the small boxes of the bug report system, while linking 
back to the PR to make it clear that a fix exists.

A week later i could prefix the PR with the issue ID which made the bot 
happy. At this point the first automated mail landed on the (correct) 
dev list which triggered the actual reviewing process - pretty cool.

suggestions:

1) I feel there should be a faster path from a PR to a JBS issue. Maybe 
there could be a command which would let the bot generate a draft issue? 
Pasting the paragraphs into the customer bug form really felt like i was 
wasting someone's time knowing that the bug reviewer will probably try 
to reproduce the bug.

2) The guide should mention somewhere that the commits will get squashed 
in the end to a single commit using the issue as title. So that the 
contributor can just keep adding commits during the review process 
without having to bother with git push -f or a clean history.

3) The bot really didn't want to add me as a contributor :) There might 
be a bug somewhere:
https://github.com/openjdk/jdk/pull/248#issuecomment-713021764

best regards,

michael

. . .

https://mbien.dev




More information about the discuss mailing list