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

update chart release action to use CR release v1.4.0 #106

Merged
merged 2 commits into from Mar 30, 2022
Merged

update chart release action to use CR release v1.4.0 #106

merged 2 commits into from Mar 30, 2022

Conversation

cpanato
Copy link
Member

@cpanato cpanato commented Mar 27, 2022

  • update chart release action to use CR release v1.4.0

@cpanato cpanato force-pushed the update140 branch 3 times, most recently from 315680b to 8188cbf Compare March 27, 2022 15:08
@cpanato
update chart release action to use CR release v1.4.0
4112b2a 
Signed-off-by: cpanato <ctadeu@gmail.com>
unguiculus
unguiculus requested changes Mar 28, 2022
View reviewed changes
.github/workflows/test-action.yml Outdated Show resolved Hide resolved
action.yml Outdated Show resolved Hide resolved
action.yml Outdated Show resolved Hide resolved
scottrigby
scottrigby reviewed Mar 28, 2022
View reviewed changes
Copy link
Member

@scottrigby scottrigby left a comment

Choose a reason for hiding this comment

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

Apart from @unguiculus suggestions, this lgtm! Once we merge this and release a new version of the action, we can update https://github.com/helm/charts-repo-actions-demo 😊

IF we want to use helm/charts-repo-actions-demo repo to test this before merging – which we have done in the past – we can do by pointing this PRs branch on your fork and ensure all is well there. Not sure if we need that anymore but it has been helpful in the past

@cpanato
update based on review
67d2272 
Signed-off-by: cpanato <ctadeu@gmail.com>
@cpanato cpanato requested a review from unguiculus March 29, 2022 07:51
@cpanato cpanato mentioned this pull request Mar 29, 2022
Merged
unguiculus
unguiculus approved these changes Mar 29, 2022
View reviewed changes
Copy link
Member

@unguiculus unguiculus left a comment

Choose a reason for hiding this comment

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

LGTM

scottrigby
scottrigby approved these changes Mar 29, 2022
View reviewed changes
Copy link
Member

@scottrigby scottrigby left a comment

Choose a reason for hiding this comment

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

🙌

@cpanato cpanato merged commit a3454e4 into helm:main Mar 30, 2022
@cpanato cpanato deleted the update140 branch March 30, 2022 08:25
renovate bot added a commit to newrelic/helm-charts that referenced this pull request Jan 23, 2024
@renovate
chore(deps): update helm/chart-releaser-action action to v1.6.0 (#1248)
c3277db 
[![Mend
Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)

This PR contains the following updates:

| Package | Type | Update | Change |
|---|---|---|---|
|
[helm/chart-releaser-action](https://togithub.com/helm/chart-releaser-action)
| action | minor | `v1.3.0` -> `v1.6.0` |

---

### Release Notes

<details>
<summary>helm/chart-releaser-action
(helm/chart-releaser-action)</summary>

###
[`v1.6.0`](https://togithub.com/helm/chart-releaser-action/releases/tag/v1.6.0)

[Compare
Source](https://togithub.com/helm/chart-releaser-action/compare/v1.5.0...v1.6.0)

#### What's Changed

- Allow to mark the published github release as latest by
[@&#8203;szaimen](https://togithub.com/szaimen) in
[helm/chart-releaser-action#135
- Support actions triggered by a tag by
[@&#8203;luisico](https://togithub.com/luisico) in
[helm/chart-releaser-action#96
- Add flag `skip-existing` to skip upload if release exists by
[@&#8203;xunleii](https://togithub.com/xunleii) in
[helm/chart-releaser-action#134
- bump to use chart-releaser v1.6.0 by
[@&#8203;cpanato](https://togithub.com/cpanato) in
[helm/chart-releaser-action#146
- Remove `charts_repo_url` from readme by
[@&#8203;viceice](https://togithub.com/viceice) in
[helm/chart-releaser-action#144
- Remove deprecated `charts_repo_url` option in documentation by
[@&#8203;krapie](https://togithub.com/krapie) in
[helm/chart-releaser-action#137
- Add `changed_charts` and `chart_version` outputs by
[@&#8203;n-oden](https://togithub.com/n-oden) in
[helm/chart-releaser-action#130
- Added: --packages-with-index parameter by
[@&#8203;valeriano-manassero](https://togithub.com/valeriano-manassero)
in
[helm/chart-releaser-action#148
- docs: add skip_existing to README by
[@&#8203;lazyfrosch](https://togithub.com/lazyfrosch) in
[helm/chart-releaser-action#158
- README: Fix default chart-releaser version by
[@&#8203;michalbiesek](https://togithub.com/michalbiesek) in
[helm/chart-releaser-action#155
- Extend `show_help` with `packages-with-index` by
[@&#8203;michalbiesek](https://togithub.com/michalbiesek) in
[helm/chart-releaser-action#160
- Add `--pages-branch` parameter by
[@&#8203;michalbiesek](https://togithub.com/michalbiesek) in
[helm/chart-releaser-action#156
- fix packages_with_index flag by
[@&#8203;cpanato](https://togithub.com/cpanato) in
[helm/chart-releaser-action#172
- bump cr to 1.6.1 and add dependabot config by
[@&#8203;cpanato](https://togithub.com/cpanato) in
[helm/chart-releaser-action#173

#### New Contributors

- [@&#8203;szaimen](https://togithub.com/szaimen) made their first
contribution in
[helm/chart-releaser-action#135
- [@&#8203;luisico](https://togithub.com/luisico) made their first
contribution in
[helm/chart-releaser-action#96
- [@&#8203;xunleii](https://togithub.com/xunleii) made their first
contribution in
[helm/chart-releaser-action#134
- [@&#8203;viceice](https://togithub.com/viceice) made their first
contribution in
[helm/chart-releaser-action#144
- [@&#8203;krapie](https://togithub.com/krapie) made their first
contribution in
[helm/chart-releaser-action#137
- [@&#8203;n-oden](https://togithub.com/n-oden) made their first
contribution in
[helm/chart-releaser-action#130
-
[@&#8203;valeriano-manassero](https://togithub.com/valeriano-manassero)
made their first contribution in
[helm/chart-releaser-action#148
- [@&#8203;lazyfrosch](https://togithub.com/lazyfrosch) made their first
contribution in
[helm/chart-releaser-action#158
- [@&#8203;michalbiesek](https://togithub.com/michalbiesek) made their
first contribution in
[helm/chart-releaser-action#155
- [@&#8203;dependabot](https://togithub.com/dependabot) made their first
contribution in
[helm/chart-releaser-action#174

**Full Changelog**:
helm/chart-releaser-action@v1.5.0...v1.6.0

###
[`v1.5.0`](https://togithub.com/helm/chart-releaser-action/releases/tag/v1.5.0)

[Compare
Source](https://togithub.com/helm/chart-releaser-action/compare/v1.4.1...v1.5.0)

#### What's Changed

- Skip packaging option - resolves
[#&#8203;108](https://togithub.com/helm/chart-releaser-action/issues/108)
by [@&#8203;gmorse81](https://togithub.com/gmorse81) in
[helm/chart-releaser-action#109
- Remove deprecated charts-repo-url option by
[@&#8203;jftanner](https://togithub.com/jftanner) in
[helm/chart-releaser-action#123
- default action to cr v1.5.0 by
[@&#8203;cpanato](https://togithub.com/cpanato) in
[helm/chart-releaser-action#127

#### New Contributors

- [@&#8203;gmorse81](https://togithub.com/gmorse81) made their first
contribution in
[helm/chart-releaser-action#109
- [@&#8203;jftanner](https://togithub.com/jftanner) made their first
contribution in
[helm/chart-releaser-action#123

**Full Changelog**:
helm/chart-releaser-action@v1...v1.5.0

###
[`v1.4.1`](https://togithub.com/helm/chart-releaser-action/releases/tag/v1.4.1)

[Compare
Source](https://togithub.com/helm/chart-releaser-action/compare/v1.4.0...v1.4.1)

#### What's Changed

- document required permissions by
[@&#8203;davidkarlsen](https://togithub.com/davidkarlsen) in
[helm/chart-releaser-action#114
- bump cr to v1.4.1 by [@&#8203;cpanato](https://togithub.com/cpanato)
in
[helm/chart-releaser-action#121

**Full Changelog**:
helm/chart-releaser-action@v1.4.0...v1.4.1

###
[`v1.4.0`](https://togithub.com/helm/chart-releaser-action/releases/tag/v1.4.0)

[Compare
Source](https://togithub.com/helm/chart-releaser-action/compare/v1.3.0...v1.4.0)

#### What's Changed

- update chart release action to use CR release v1.4.0 by
[@&#8203;cpanato](https://togithub.com/cpanato) in
[helm/chart-releaser-action#106

**Full Changelog**:
helm/chart-releaser-action@v1.3.0...v1.4.0

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined),
Automerge - At any time (no schedule defined).

🚦 **Automerge**: Enabled.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the
rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update
again.

---

- [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check
this box

---

This PR has been generated by [Mend
Renovate](https://www.mend.io/free-developer-tools/renovate/). View
repository job log
[here](https://developer.mend.io/github/newrelic/helm-charts).

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNy4xMzUuMCIsInVwZGF0ZWRJblZlciI6IjM3LjEzNS4wIiwidGFyZ2V0QnJhbmNoIjoibWFzdGVyIn0=-->

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@scottrigby scottrigby scottrigby approved these changes

@unguiculus unguiculus unguiculus approved these changes

@davidkarlsen davidkarlsen Awaiting requested review from davidkarlsen

Assignees
No one assigned
Labels
size/L
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@cpanato @scottrigby @unguiculus