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

feat: add pod namespace as tag in dtrack project #157

Merged
merged 2 commits into from
Aug 29, 2022
Merged

feat: add pod namespace as tag in dtrack project #157

merged 2 commits into from
Aug 29, 2022

Conversation

rvgud
Copy link
Contributor

@rvgud rvgud commented Aug 23, 2022

This PR is to add the namespace name as tag in project of dtrack

Fixes: #156

This PR is to add the namespace name as tag in project of dtrack

Fixes: #156
@github-actions github-actions bot added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label Aug 23, 2022
Copy link
Owner

@ckotzbauer ckotzbauer left a comment

Choose a reason for hiding this comment

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

Hi @rvgud,
thanks for the request and the PR, this is very appreciated!
The ProcessSbom function is an interface of type Target. You have to add the parameter to the interface and other implementations too (git, oci)

@ckotzbauer ckotzbauer added the kind/feature Categorizes issue or PR as related to a new feature. label Aug 27, 2022
@rvgud rvgud requested a review from ckotzbauer August 29, 2022 07:15
@ckotzbauer ckotzbauer merged commit 11ba433 into ckotzbauer:main Aug 29, 2022
@ckotzbauer
Copy link
Owner

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/feature Categorizes issue or PR as related to a new feature. size/S Denotes a PR that changes 10-29 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

feature-request - add namespace name as tag in dtrack project
2 participants