-
Notifications
You must be signed in to change notification settings - Fork 83
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
adding release process document #211
Conversation
Signed-off-by: Aaron Schlesinger <aaron@ecomaz.net>
Signed-off-by: Aaron Schlesinger <aaron@ecomaz.net>
Signed-off-by: Aaron Schlesinger <aaron@ecomaz.net>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thanks!
|
||
To determine the new version, follow [SemVer guidelines](https://semver.org). Most releases will increment the PATCH or MINOR version number. | ||
|
||
## 1: Submit a PR to the [Helm Charts Repository](https://github.com/kedacore/charts) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'd move this to step 2 since there is no new app version yet.
Worst case the container was shipped but the Chart PR is still in review.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@arschles Automerge ignores comments, would you mind taking a look at this comment and share what you think?
* adding release process document Signed-off-by: Aaron Schlesinger <aaron@ecomaz.net> * updating helm chart and adding docs/blog Signed-off-by: Aaron Schlesinger <aaron@ecomaz.net> * fixing links Signed-off-by: Aaron Schlesinger <aaron@ecomaz.net>
This PR adds a
RELEASE-PROCESS.md
document that instructs maintainers on how to release a new version of the HTTP add-on.Checklist
README.md
docs/
directoryFixes #195
cc/ @tomkerkhove