fix(workflow): move chart release workflow and fix fetch-depth bug

Yuni-sa commented 4 months ago

What this PR does / why we need it?

Moved the Helm chart release steps from release.yml to a new chart-release.yml workflow for better separation of concerns.
Fixed a bug where the chart-releaser-action wouldn't find changes by adding fetch-depth: 0 in the checkout step.

PR Checklist

[ ] This PR adds K8s exceptions (false positives)
[ ] This PR adds new code
[ ] This PR includes tests for new/existing code
[ ] This PR adds docs
[X] This PR changes the project workflow

GitHub Issue

Closes #325

Notes for your reviewers

Moved the Helm chart release steps from release.yml to chart-release.yml for better separation of concerns and to avoid slowing down the main CI process with fetch-depth: 0.

codecov-commenter commented 4 months ago

:warning: Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 42.01%. Comparing base (181141d) to head (1ec6858).

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files

```diff @@ Coverage Diff @@ ## main #326 +/- ## ======================================= Coverage 42.01% 42.01% ======================================= Files 61 61 Lines 3175 3175 ======================================= Hits 1334 1334 Misses 1633 1633 Partials 208 208 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

yonahd commented 4 months ago

@Yuni-sa I added a comment. If we're good let's merge this and test

yonahd / kor