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

add pre-commit and replace black with ruff-format #339

Merged
merged 1 commit into from
Jul 21, 2024
Merged

Conversation

gruebel
Copy link
Collaborator

@gruebel gruebel commented Jul 21, 2024

What does this PR do?

  • streamlining with policy_sentry setup

What gif best describes this PR or how it makes you feel?

Completion checklist

  • Additions and changes have unit tests
  • The pull request has been appropriately labeled using the provided PR labels
  • GitHub actions automation is passing (make test, make lint, make security-test, make test-js)
  • If the UI contents or JavaScript files have been modified, generate a new example report:
# Generate the updated Javascript bundle
make build-js

# Generate the example report
make generate-report

@gruebel gruebel added the automation CI/CD and whatnot label Jul 21, 2024
@gruebel gruebel merged commit ab180f3 into master Jul 21, 2024
13 checks passed
@gruebel gruebel deleted the pre-commit branch July 21, 2024 16:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automation CI/CD and whatnot cla:signed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant