Skip to content

Refactor scanner HelmCharts to introduce consistency in HelmChart Values #483

Merged
J12934 merged 20 commits intosecureCodeBox:v3from
EndPositive:maintanance/helmchart-values-consistency-scanners
Jun 9, 2021
Merged

Refactor scanner HelmCharts to introduce consistency in HelmChart Values #483
J12934 merged 20 commits intosecureCodeBox:v3from
EndPositive:maintanance/helmchart-values-consistency-scanners

Conversation

@EndPositive
Copy link
Contributor

@EndPositive EndPositive commented Jun 9, 2021

Description

This PR brings more consistency to scanner HelmChart Values according to the spec described in #472.

Checklist

  • Test your changes as thoroughly as possible before you commit them. Preferably, automate your test by unit/integration tests.
  • Make sure npm test runs for the whole project.
  • Make codeclimate checks happy

Signed-off-by: Jop Zitman <jop-zitman@hotmail.com>
Signed-off-by: Jop Zitman <jop-zitman@hotmail.com>
Signed-off-by: Jop Zitman <jop-zitman@hotmail.com>
Signed-off-by: Jop Zitman <jop-zitman@hotmail.com>
Signed-off-by: Jop Zitman <jop-zitman@hotmail.com>
Signed-off-by: Jop Zitman <jop-zitman@hotmail.com>
Signed-off-by: Jop Zitman <jop-zitman@hotmail.com>
Signed-off-by: Jop Zitman <jop-zitman@hotmail.com>
Signed-off-by: Jop Zitman <jop-zitman@hotmail.com>
…esTable to gotmpl

Signed-off-by: Jop Zitman <jop-zitman@hotmail.com>
Signed-off-by: Jop Zitman <jop-zitman@hotmail.com>
Signed-off-by: Jop Zitman <jop-zitman@hotmail.com>
Signed-off-by: Jop Zitman <jop-zitman@hotmail.com>
Signed-off-by: Jop Zitman <jop-zitman@hotmail.com>
Signed-off-by: Jop Zitman <jop-zitman@hotmail.com>
Signed-off-by: Jop Zitman <jop-zitman@hotmail.com>
Signed-off-by: Jop Zitman <jop-zitman@hotmail.com>
Signed-off-by: Jop Zitman <jop-zitman@hotmail.com>
Signed-off-by: Jop Zitman <jop-zitman@hotmail.com>
@EndPositive EndPositive marked this pull request as ready for review June 9, 2021 13:35
@EndPositive
Copy link
Contributor Author

Should pass the integration tests. Is it possible to trigger them on this PR?

@J12934
Copy link
Member

J12934 commented Jun 9, 2021

Not easily at the Moment. @fuhrmeistery is working on restructuring our ci so that it can run for external contributors.

You can run the pipelines on your fork, but this won't mark them as passed in the pull request: https://github.com/secureCodeBox/secureCodeBox/blob/main/CONTRIBUTING.md#working-with-forks-and-pull-requests

I'll take a look at the PR and merge it to a v3 branch in the repo, so the checks can run.

@J12934 J12934 added breaking Changes requiring a major release maintenance labels Jun 9, 2021
@J12934 J12934 added this to the v3.0.0 milestone Jun 9, 2021
@J12934 J12934 self-requested a review June 9, 2021 19:25
@J12934 J12934 changed the base branch from main to v3 June 9, 2021 19:25
Copy link
Member

@J12934 J12934 left a comment

Choose a reason for hiding this comment

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

💯

@J12934 J12934 merged commit 2146519 into secureCodeBox:v3 Jun 9, 2021
@EndPositive EndPositive deleted the maintanance/helmchart-values-consistency-scanners branch June 9, 2021 20:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

breaking Changes requiring a major release maintenance

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants