[NeXus-committee] Pull request etiquette

Paul Millar paul.millar at desy.de
Tue Oct 22 17:53:03 BST 2024


Just FYI,

Information on how people may contribute to a project is (by convention) 
often recorded in the 'CONTRIBUTING' or 'CONTRIBUTING.md' file (if 
markdown is used).  This is a file stored in the repo's root directory, 
much like README / README.md.

Amongst other things, the 'CONTRIBUTING.md' file often describes what is 
expected when proposing a change.  So, a policy requiring that an issue 
is created first would (I think) naturally be documented in that file.

HTH,
Paul.

On 22/10/2024 02.05, Aaron Brewster via NeXus-committee wrote:
> Thirded!  For all the above reasons.
>
> On Fri, Oct 18, 2024 at 10:30 AM Pete Jemian via NeXus-committee 
> <nexus-committee at shadow.nd.rl.ac.uk> wrote:
>
>     I second this as the minimum requirement. Every PR should
>     reference an issue.
>
>     Issues describe "why" this is important.
>
>     PRs describe "how" the issue is handled.
>
>     The most expensive question is "why". In current practice, this
>     context is omitted far too often.
>
>
>     On Fri, Oct 18, 2024, 11:38 AM Osborn, Raymond via NeXus-committee
>     <nexus-committee at shadow.nd.rl.ac.uk> wrote:
>
>         I would like to suggest that we require all pull requests to
>         have a detailed description of what is being proposed and why.
>         It is becoming impossible to understand the context of many of
>         the PRs, whose conversations fill my inbox every day.
>         Sometimes there are links to other related PRs or issues, but
>         these often contain no description either.
>
>         I think the assumption is that these are things that were
>         either discussed at the recent NIAC or in a telco, but even if
>         we were present, it can be difficult to remember them all and
>         those who weren’t present have no chance of knowing what is
>         going on. Detailed descriptions are also important as a
>         historical record of why we make the decisions that we do.
>
>         Ray
>
>         -- 
>         Ray Osborn, Senior Scientist
>         Materials Science Division
>         Argonne National Laboratory
>         Lemont, IL 60439, USA
>         Phone: +1 (630) 252-9011
>         Email: ROsborn at anl.gov
>
>         _______________________________________________
>         NeXus-committee mailing list
>         NeXus-committee at nexusformat.org
>         https://lists.nexusformat.org/mailman/listinfo/nexus-committee
>
>     _______________________________________________
>     NeXus-committee mailing list
>     NeXus-committee at nexusformat.org
>     https://lists.nexusformat.org/mailman/listinfo/nexus-committee
>
>
> _______________________________________________
> NeXus-committee mailing list
> NeXus-committee at nexusformat.org
> https://lists.nexusformat.org/mailman/listinfo/nexus-committee



More information about the NeXus-committee mailing list