Skip to content
This repository has been archived by the owner on Jan 18, 2024. It is now read-only.

Update README.md #540

Merged
merged 1 commit into from
Jan 3, 2023
Merged

Update README.md #540

merged 1 commit into from
Jan 3, 2023

Conversation

alejandrodnm
Copy link
Contributor

@alejandrodnm alejandrodnm commented Jan 3, 2023

Signed-off-by: alejandrodnm alejandrodnm@gmail.com

What this PR does / why we need it

Which issue this PR fixes

  • fixes README

Special notes for your reviewer

Checklist

@alejandrodnm alejandrodnm requested a review from a team as a code owner January 3, 2023 11:53
Copy link
Contributor

@onprem onprem left a comment

Choose a reason for hiding this comment

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

Thanks! Got caught by you again haha.

@onprem
Copy link
Contributor

onprem commented Jan 3, 2023

You'd need to bump the chart version though, for how our CI works.

@alejandrodnm
Copy link
Contributor Author

@onprem do I need to bump the chart version just for this change?

@onprem
Copy link
Contributor

onprem commented Jan 3, 2023

@alejandrodnm Yes, we have CD setup to release a new version of chart with every PR, if there any changes in it's files. Since the README file is under the chart directory, we need to bump the version. Here we can just do a patch release.

Signed-off-by: alejandrodnm <alejandrodnm@gmail.com>
@alejandrodnm
Copy link
Contributor Author

@onprem the errors are related to a fix for the alerts that was made in this PR timescale/promscale#1794 Is there something else I need to do?

@alejandrodnm
Copy link
Contributor Author

I'll leave this open until the alerts are fixed on another PR

@onprem
Copy link
Contributor

onprem commented Jan 3, 2023

I have created #541 for the alerts. And these checks are unrelated and not required for this PR so I am going ahead and merging. Thanks!

@onprem onprem merged commit 41ec765 into main Jan 3, 2023
@onprem onprem deleted the alejandrodnm-patch-1 branch January 3, 2023 12:47
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants