Hello, I would like to suggest a couple of changes to the Pull Request template.
The release checklist currently has: - [ ] Authors updated in cf-conventions.adoc? - [ ] Next version in cf-conventions.adoc up to date? Versioning inspired by SemVer. - [ ] history.adoc up to date? - [ ] Conformance document up-to-date? I propose removing two items, changing it to - [ ] Authors updated in cf-conventions.adoc? - [ ] Conformance document up-to-date? These changes reflect the fact that * The version may or may not have already been incremented, and in any case it is beyond the scope of the PR. * The history entry should include the date when the PR was merged, which is unknown to the proposer. The history should instead be updated by the merger, and in any event the history checked prior to a release. Perhaps we chould have two checklists - one for the Author and one for the Merger - that might look like: #### Author checklist - [ ] Authors updated in cf-conventions.adoc? - [ ] Conformance document up-to-date? #### Merger checklist - [ ] history.adoc up to date? Thoughts welcome! -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://urldefense.us/v3/__https://github.com/cf-convention/cf-conventions/issues/325__;!!G2kpM7uM-TzIFchu!mhMVz908DObfit-wHj2wkc-GoiTguuFDmW08ML2EIZi76wK89ZbKJGqWPRrelLO-T9h2HUGstsk$ This list forwards relevant notifications from Github. It is distinct from cf-metad...@cgd.ucar.edu, although if you do nothing, a subscription to the UCAR list will result in a subscription to this list. To unsubscribe from this list only, send a message to cf-metadata-unsubscribe-requ...@listserv.llnl.gov.
