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.

Sign up for GitHub

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

Jump to bottom

Issue 763 #1090

Merged
merged 11 commits into from
May 20, 2024
Merged

Issue 763 #1090

merged 11 commits into from
May 20, 2024

Conversation

cerrussell
Copy link
Contributor

@cerrussell cerrussell commented May 17, 2024 •
edited

Adds generation of bom snapshots and testing of these snapshots. Additional bom repos will be added in future to increase coverage.

Closes #763

@prabhu
Copy link
Contributor

prabhu commented May 18, 2024

@cerrussell, what is left on this ticket?

@cerrussell
Add script to bulk generate sample boms for initial sample repos for …
538a599 
…testing purposes.

Signed-off-by: Caroline Russell <caroline@appthreat.dev>
@cerrussell
Add workflow to test bom snapshots.
57f9fb0 
Signed-off-by: Caroline Russell <caroline@appthreat.dev>
@cerrussell
Refine bom creation.
0bd3ba5 
Signed-off-by: Caroline Russell <caroline@appthreat.dev>
@cerrussell
Create initial snapshot boms.
2c773ba 
Signed-off-by: Caroline Russell <caroline@appthreat.dev>
@cerrussell
Create initial snapshot tests.
c9564fe 
Signed-off-by: Caroline Russell <caroline@appthreat.dev>
@cerrussell
Update snapshot testing workflow.
e43c1d6 
Signed-off-by: Caroline Russell <caroline@appthreat.dev>
@cerrussell
Move bom snapshots to separate repo.
c5cc7d2 
Signed-off-by: Caroline Russell <caroline@appthreat.dev>
@cerrussell
Add a few more repos.
8d57fd8 
Signed-off-by: Caroline Russell <caroline@appthreat.dev>
@cerrussell
Refine bom snapshot generation.
1c449f6 
Signed-off-by: Caroline Russell <caroline@appthreat.dev>
@cerrussell
Make snapshot tests for CI.
8509c91 
Signed-off-by: Caroline Russell <caroline@appthreat.dev>
@cerrussell cerrussell marked this pull request as ready for review May 20, 2024 02:43
@cerrussell
Codacy fix for import subprocess
79880e6 
Signed-off-by: Caroline Russell <caroline@appthreat.dev>
@cerrussell
Copy link
Contributor Author

@prabhu Nothing.

prabhu
prabhu approved these changes May 20, 2024
View reviewed changes
Copy link
Contributor

@prabhu prabhu left a comment

Choose a reason for hiding this comment

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

Let's do this!

@prabhu prabhu merged commit 4e6f8da into master May 20, 2024
23 checks passed
@prabhu prabhu deleted the fix/issue-763 branch May 20, 2024 15:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@prabhu prabhu prabhu approved these changes

@setchy setchy Awaiting requested review from setchy setchy is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[QA] Snapshot testing
2 participants
@cerrussell @prabhu