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: use custom pre commit config in refactor #479

Conversation

JordoHeffernan
Copy link
Contributor

secureli-XXX

including recent feature from main branch to maintain settings if repo has existing pre-commit-config.yaml on secureli init.

Changes

*Updates to pre-commit, action, scan, and language support to check for existing congif yaml, and copy relevant contents into newly generated .secureli/pre-commit-config.yaml as well as lots of new tests.

Testing

Clean Code Checklist

  • Meets acceptance criteria for issue
  • New logic is covered with automated tests
  • Appropriate exception handling added
  • Thoughtful logging included
  • Documentation is updated
  • Follow-up work is documented in TODOs
  • TODOs have a ticket associated with them
  • No commented-out code included

@JordoHeffernan JordoHeffernan merged commit 7cb0a6b into refactor/secureli-000-modular-refactor Mar 18, 2024
3 checks passed
@JordoHeffernan JordoHeffernan deleted the feat/secureli-373-use-custom-pre-commit-config-refactor branch March 18, 2024 20:24
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.

2 participants