Skip to content
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

Update dependencies as per release-process.md. #5766

Merged
merged 2 commits into from
Dec 5, 2022

Conversation

absoludity
Copy link
Contributor

@absoludity absoludity commented Dec 1, 2022

Everything up to 0.5 in the release-process.md

Note: there is not a lot to review here: the only manual changes are those to the release-process.md itself and a couple of vars that were no longer used.

Signed-off-by: Michael Nelson [email protected]

Description of the change

Benefits

Check and get ready for release.

Possible drawbacks

Applicable issues

  • fixes #

Additional information

@netlify
Copy link

netlify bot commented Dec 1, 2022

Deploy Preview for kubeapps-dev canceled.

Name Link
🔨 Latest commit 8ce4cb1
🔍 Latest deploy log https://app.netlify.com/sites/kubeapps-dev/deploys/638d289af396640008f013bd

@@ -34,12 +34,11 @@ env:
KUBEAPPS_REPO: "vmware-tanzu/kubeapps"
BRANCH_KUBEAPPS_REPO: "main"
README_GENERATOR_REPO: "bitnami-labs/readme-generator-for-helm"
DOCKER_VERSION: "20.10.18"
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This var seems to be no longer used, I removed the mention in the release-process.md too @beni0888 , just double check that, and the other changes I made to release-process.md are OK in your eyes :)

Comment on lines -65 to +66
$ref: "#/components/schemas/v1alpha1GetAvailablePackageSummariesResponse"
$ref: "#/components/schemas/v1alpha1GetAvailablePackageSummariesRes\
ponse"
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The main changes (other than the actual ones) to the openapi.yaml appeared to be simple formatting like the above. Not sure why but assume that's OK @antgamdia ?

Everything up to 0.5

Signed-off-by: Michael Nelson <[email protected]>
Signed-off-by: Michael Nelson <[email protected]>
@absoludity
Copy link
Contributor Author

I'm going to merge this PR so that I can move forward with the release prep (and it requires conflict resolution every time we land a bump dependabot PR). Happy to change anything as needed.

@absoludity absoludity merged commit a906b91 into main Dec 5, 2022
@absoludity absoludity deleted the release-update-dev-images branch December 5, 2022 01:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants