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

Provide secret scanning for local setup #963

Merged

Conversation

serverhorror
Copy link
Contributor

@serverhorror serverhorror commented Oct 25, 2023

Description here ...

Closes #...
Fixes #...

Tasks:

  • Updated documentation in docs/modules/... directory
  • Ran tests in <quickstarter>/testdata directory

@serverhorror
Copy link
Contributor Author

There we go, as dicussed in our meeting.

This:

  • starts out with gitleaks
  • has no impact on current usage (if not activated by the user)
  • gives us the structure
  • if everything works out we have a central partial that will allow to update the docs in a central place

Anything that I'm missing?

@serverhorror serverhorror marked this pull request as ready for review October 25, 2023 11:17
gerardcl
gerardcl previously approved these changes Oct 25, 2023
Copy link
Member

@gerardcl gerardcl left a comment

Choose a reason for hiding this comment

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

LGTM! 🚀

Copy link
Member

@cschweikert cschweikert left a comment

Choose a reason for hiding this comment

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

Good stuff! Thanks for introducing this 😊

CHANGELOG.md Outdated Show resolved Hide resolved
Co-authored-by: Christian Schweikert <[email protected]>
@serverhorror
Copy link
Contributor Author

Hello,

@michaelsauter, @oalyman, @metmajer, @nichtraunzer, @bljubisic, @buegelbeatz, @tbugfinder, @sino92, @clemensutschig, @sklingel

Your reviews are missing. Can this be merged? Are there any tests that need to run for this to be accepted?

Copy link
Member

@michaelsauter michaelsauter left a comment

Choose a reason for hiding this comment

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

Sorry wasn't aware you're waiting for approval from everyone :) This is great, please go ahead!

@serverhorror
Copy link
Contributor Author

Sorry wasn't aware you're waiting for approval from everyone :) This is great, please go ahead!

I am not, I will never merge my own PRs. Apart from that: I don't even have the technical permissions.

Whoever merges, merges :)

@serverhorror
Copy link
Contributor Author

@metmajer, @nichtraunzer, @tbugfinder, @bljubisic, @sklingel, @sino92, @buegelbeatz, @clemensutschig

Ping!

Your approvals are still missing, or you can merge if you are happy with the existing approvals.

@gerardcl gerardcl merged commit 832c9d8 into opendevstack:master Nov 16, 2023
10 checks passed
@serverhorror serverhorror deleted the feature/enable-secret-scanning branch November 23, 2023 21:49
@BraisVQ BraisVQ mentioned this pull request Apr 22, 2024
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.