Skip to content

Add release notes to GH releases#581

Open
morganchristiansson wants to merge 2 commits into
influxdata:masterfrom
morganchristiansson:add-release-notes
Open

Add release notes to GH releases#581
morganchristiansson wants to merge 2 commits into
influxdata:masterfrom
morganchristiansson:add-release-notes

Conversation

@morganchristiansson

@morganchristiansson morganchristiansson commented Jul 14, 2023

Copy link
Copy Markdown

I noticed grafana/helm-charts repo has started to include changelog on releases and would really like to see the same in this repo!

Test release with changelog in my forked repo: https://github.com/morganchristiansson/influxdata-charts/releases/tag/telegraf-1.8.34

Changes copied from commit: grafana/helm-charts@98f267b

Also upgraded helm version - copied from this commit grafana/helm-charts@8d26baa

@morganchristiansson

Copy link
Copy Markdown
Author

helm 3.12.2 is out can maybe upgrade to latest. https://github.com/helm/helm/releases

I don't think helm upgraded is required tho, as you prefer.

@powersj powersj left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

No opposed, but can the cr.yaml file live under the .github/workflows directory as well?

@morganchristiansson

Copy link
Copy Markdown
Author

Isn't .github/workflows reserved for github workflows? doesn't seem like the right place.

There is already a ct.yaml (used by chart-testing) in root so this would go next to it.

Actually seems like cr.yaml in current directory is loaded by default so with: config: cr.yaml could be omitted.

@powersj

powersj commented Sep 7, 2023

Copy link
Copy Markdown
Contributor

Isn't .github/workflows reserved for github workflows? doesn't seem like the right place.

This file is referenced by the github workflow, hence why I thought it should be in there. And avoids adding a file to the root of the project. This is not something I would expect someone to run by hand right?

@morganchristiansson

Copy link
Copy Markdown
Author

chart-releaser runs great locally from CLI but you probably wouldn't except for debug/setup.

I'm not sure adding new .yaml files to .github/workflows will interfere with GitHub Actions? I'm happy to make the change if wanted tho...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants