apache / accumulo-website

Apache Accumulo Website
https://accumulo.apache.org
14 stars 100 forks source link

Improvements to 2.1.2 release notes #393

Closed DomGarguilo closed 1 year ago

DomGarguilo commented 1 year ago

This PR:

ctubbsii commented 1 year ago

The discussion Ed was referring to wasn't a public discussion... just me privately expressing my opinion that I think updates to draft release notes don't really need a PR. As long as it's properly marked as draft, and doesn't break the website, I think updating its content is something you can just lean on the "commit then review" mindset more than the "PR just in case" mindset. You can still create PRs if you want... but the draft release notes are going to receive a bunch of updates/polishing/review before the draft label gets removed anyway. So, unless you want somebody to specifically check your wording for technical correctness, it's probably not a problem to just commit it directly.