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 versions #157

Merged
merged 2 commits into from
May 7, 2024
Merged

Update versions #157

merged 2 commits into from
May 7, 2024

Conversation

wendigo
Copy link
Contributor

@wendigo wendigo commented May 7, 2024

No description provided.

@cla-bot cla-bot bot added the cla-signed label May 7, 2024
@wendigo wendigo requested a review from mosabua May 7, 2024 19:31
@wendigo wendigo requested a review from losipiuk May 7, 2024 19:53

# This is the version number of the application being deployed. This version number should be
# incremented each time you make changes to the application. Versions are not expected to
# follow Semantic Versioning. They should reflect the version the application is using.
# Same value as in values.yml#image.tag
appVersion: "432"
Copy link
Member

Choose a reason for hiding this comment

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

Cool... with the next Java update we have to think about updating the jvm config .. or hold back on the chart for a while

Copy link
Member

@mosabua mosabua left a comment

Choose a reason for hiding this comment

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

Looks good. Are we okay to cut a release in terms of the merged and waiting PRs?

@wendigo
Copy link
Contributor Author

wendigo commented May 7, 2024

@mosabua i've already created a tag but I don't know if this is enough to release. Please advise

@mosabua
Copy link
Member

mosabua commented May 7, 2024

You just merge .., the rest happens automatically

@mosabua mosabua merged commit a3a62a1 into main May 7, 2024
6 checks passed
@mosabua mosabua deleted the serafin/0.20.0 branch May 7, 2024 20:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

4 participants