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(all): Replace trufflehog with detect-secrets #983

Merged
merged 12 commits into from
Jan 13, 2025

Conversation

ElenaKhaustova
Copy link
Contributor

@ElenaKhaustova ElenaKhaustova commented Jan 10, 2025

Description

Solves kedro-org/kedro#4062 for plugins

In this PR we replace trufflehog with detect-secrets - https://github.com/Yelp/detect-secrets for secrets detecting locally and at the CI.

Extended validate-pr-title.yaml scopes with all to handle cases when changes are applied to all plugins.

Development notes

Please see notes for the similar PR for the framework: kedro-org/kedro#4404

Example of check failing for plugin at the CI: https://github.com/kedro-org/kedro-plugins/actions/runs/12745897204/job/35520826972?pr=983

Checklist

  • Opened this PR as a 'Draft Pull Request' if it is work-in-progress
  • Updated the documentation to reflect the code changes
  • Updated jsonschema/kedro-catalog-X.XX.json if necessary
  • Added a description of this change in the relevant RELEASE.md file
  • Added tests to cover my changes
  • Received approvals from at least half of the TSC (required for adding a new, non-experimental dataset)

Signed-off-by: Elena Khaustova <[email protected]>
Signed-off-by: Elena Khaustova <[email protected]>
Signed-off-by: Elena Khaustova <[email protected]>
@ElenaKhaustova ElenaKhaustova changed the title Replace trufflehog with detect-secrets feat(airflow/datasets/docker/telemetry): Replace trufflehog with detect-secrets Jan 10, 2025
@ElenaKhaustova ElenaKhaustova changed the title feat(airflow/datasets/docker/telemetry): Replace trufflehog with detect-secrets feat(all): Replace trufflehog with detect-secrets Jan 10, 2025
@ElenaKhaustova ElenaKhaustova marked this pull request as ready for review January 13, 2025 11:34
Copy link
Contributor

@ankatiyar ankatiyar left a comment

Choose a reason for hiding this comment

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

LGTM! 👍🏾

Signed-off-by: Elena Khaustova <[email protected]>
Copy link
Member

@merelcht merelcht left a comment

Choose a reason for hiding this comment

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

Thanks @ElenaKhaustova, looks good 👍

Signed-off-by: Elena Khaustova <[email protected]>
@ElenaKhaustova ElenaKhaustova enabled auto-merge (squash) January 13, 2025 16:31
@ElenaKhaustova ElenaKhaustova merged commit bf0c407 into main Jan 13, 2025
51 checks passed
@ElenaKhaustova ElenaKhaustova deleted the feature/replace-trufflehog branch January 13, 2025 16:55
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.

3 participants