vinzscam / backstage-chart

Backstage Helm Chart
27 stars 10 forks source link

Adds Github Actions for PR workflow #26

Closed ChrisJBurns closed 1 year ago

ChrisJBurns commented 1 year ago

As described in https://github.com/vinzscam/backstage-chart/issues/21 we do not currently have any automated tests triggered when we make Helm Chart changes. This PR adds a simple lint and e2e test for the Backstage chart, so that we at least know if we make any breaking changes that stop the Chart installation - further reducing the need for manual efforts.

An additional point, it will also fail the PR Github Workflow if the Chart version hasn't been bumped up - this reduces the need for manual commits later on to bump the Chart versions.

Signed-off-by: ChrisJBurns 29541485+ChrisJBurns@users.noreply.github.com

ChrisJBurns commented 1 year ago

@vinzscam @cmoulliard Welcome your thoughts on the above. It currently doesn't run any Linting because it hasn't detected a change with the Charts.

ChrisJBurns commented 1 year ago

Closing as this repo has been deprecated in favour of https://github.com/backstage/charts