Skip to content

Cascading scans codeclimate#482

Merged
J12934 merged 3 commits intosecureCodeBox:mainfrom
EndPositive:maintanance/cascading-scans-codeclimate
Jun 9, 2021
Merged

Cascading scans codeclimate#482
J12934 merged 3 commits intosecureCodeBox:mainfrom
EndPositive:maintanance/cascading-scans-codeclimate

Conversation

@EndPositive
Copy link
Copy Markdown
Contributor

Description

This PR resolves the codeclimate issues that the cascading scan hook was having.

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

…itive complexity

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

Signed-off-by: Jop Zitman <jop-zitman@hotmail.com>
Signed-off-by: Jop Zitman <jop-zitman@hotmail.com>
@EndPositive EndPositive changed the title Maintanance/cascading scans codeclimate Cascading scans codeclimate Jun 9, 2021
@J12934 J12934 self-requested a review June 9, 2021 19:14
Copy link
Copy Markdown
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.

Nice 🚀
Will merge this on mainas this is a refactoring which doesn't change the hooks behavior.

@J12934 J12934 merged commit 7367c12 into secureCodeBox:main Jun 9, 2021
@EndPositive EndPositive deleted the maintanance/cascading-scans-codeclimate branch June 9, 2021 20:41
@rfelber rfelber added this to the 2.9.0 milestone Jun 18, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants