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.

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

ci: add version comparison workflow #1350

Merged
merged 1 commit into from
Oct 24, 2023

Conversation

didroe
Copy link
Contributor

@didroe didroe commented Oct 23, 2023

Description

Adds a new manual workflow for comparing a test version of the CLI/rules against a baseline version. The security report is generated as JSON for both versions, and then a diff of the reports is outputted. If there are differences then the workflow fails.

The base/test CLI/rules versions can be specified as a git tag, branch name, or commmit SHA. Leaving a ref blank uses the latest commit on the default branch.

Also synchronizes the KPI repository list with those in the battle test repo.

Checklist

  • I've added test coverage that shows my fix or feature works as expected.
  • I've updated or added documentation if required.
  • I've included usage information in the description if CLI behavior was updated or added.
  • PR title follows Conventional Commits format

@didroe didroe force-pushed the chore/add-version-comparison-workflow branch 18 times, most recently from bdd70bc to ed6928a Compare October 24, 2023 10:04
@didroe didroe force-pushed the chore/add-version-comparison-workflow branch from ed6928a to 99c32fe Compare October 24, 2023 12:41
@didroe didroe marked this pull request as ready for review October 24, 2023 12:41
@didroe didroe requested a review from gotbadger October 24, 2023 14:11
@didroe didroe merged commit 9a61366 into main Oct 24, 2023
8 checks passed
@didroe didroe deleted the chore/add-version-comparison-workflow branch October 24, 2023 15:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants