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
Adding detector for Nvidia NGC #797
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thanks a lot for the contribution 😄
Hey @ahrav , I'm not bumping this in priority, but I want to understand the path from here. Is it just as simple as you merge whenever you're ready? Or is there more involved with transitioning the test secrets to a trufflehog-owned secrets manager + anything else? |
My apologies I completely missed this one. We actually do not need to transition the secrets on our end. So long as the tests had passed during your run that is sufficient for us. I think there might've been new detectors added recently which is why there looks to be a merge conflict. Once that is resolved feel free to merge this one in. Again apologies for the delayed response here. |
This is awesome - thank you @JosephTLucas !!! |
Thank you @JosephTLucas |
Reference: #790