 |
 |
Hi, I’d like to request that everyone put a bit more detailed information into their hps-java Pull Requests from now on. Firstly, please provide a descriptive title other than the automatically created one (e.g. ‘Iss1234’) which says exactly what the PR does so the reviewer doesn’t need to try and figure it out themselves. Just listing the issue number in the title is not that helpful as the issue branch to be merged will already be linked automatically from the PR, so the title should actually say what you are doing. Secondly, write a paragraph or two in the PR description (as needed) to describe in detail the changes you are making, especially for PR with many changes. A bulleted list can be useful if there are a number of disparate items that you have changed on the branch. Thanks. —Jeremy ######################################################################## Use REPLY-ALL to reply to list To unsubscribe from the HPS-SOFTWARE list, click the following link: https://listserv.slac.stanford.edu/cgi-bin/wa?SUBED1=HPS-SOFTWARE&A=1