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.

Sign up for GitHub

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

Jump to bottom

updated helm note to add namespace, closes #422 #423

Merged
merged 1 commit into from Aug 2, 2023
Merged

updated helm note to add namespace, closes #422 #423

merged 1 commit into from Aug 2, 2023

Conversation

EvanCarroll
Copy link
Contributor

This is just a note upgrade, documented in #422

@provokateurin
Copy link
Member

Version bump is required

@jessebot
Copy link
Collaborator

jessebot commented Aug 2, 2023 •
edited

@EvanCarroll , Kate is referring to the version number here, which needs to have the patch number (the last one) bumped:

helm/charts/nextcloud/Chart.yaml

Line 3 in ba866ea

version: 3.5.20

After that, one of the collaborators/maintainers can approve the workflow and run it to make sure it checks out before we merge it.

@EvanCarroll
Updated helm note to add namespace, closes nextcloud#422
d6d1371 
Signed-off-by: Evan Carroll <me@evancarroll.com>
@EvanCarroll
Copy link
Contributor Author

@provokateurin @jessebot done

@provokateurin provokateurin merged commit e9201c7 into nextcloud:main Aug 2, 2023
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@provokateurin provokateurin provokateurin approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@EvanCarroll @provokateurin @jessebot