Kong / kong-operator

Kong Operator for Kubernetes and OpenShift
https://konghq.com
Apache License 2.0
58 stars 27 forks source link

chore: define CONTRIBUTING.md with release instructions #43

Closed mflendrich closed 4 years ago

mflendrich commented 4 years ago

closes #23

Not to be merged before #38 because the instructions from CONTRIBUTING.md expect the artifact (introduced in #38) to be present on CI runs.

mflendrich commented 4 years ago

The release instructions are based on my experience from releasing v0.4.0. We can battle-test the instructions with our next release.

Also, these instructions come with a lot of potential for automation. See #21 #40 . We've made the choice to start with a manual (yet well-defined) release workflow that we can go through several times before we automate.

rainest commented 4 years ago

Generally looks good. Two things I'd like to raise:

mflendrich commented 4 years ago

does it make sense to formalize that as "do the release in 5, but use the GH Pre-release feature to mark it as an RC" and then "assuming nothing went wrong and everything is merged upstream, mark the release gold after 10 is done"

I would say "no" because at that point the release is already validated by CI, and there's no (apparent to me) scope for validation happening after that step.

Has there been any broader discussion around targeting some standard as the eventual standard across all our OSS projects?

No, my thinking was that leading by example (+ the right announcement) could be more effective. Also, defragmentation of contributing guidelines is imo out of scope.

mflendrich commented 4 years ago

@rainest can we merge this now?