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

fix(rules): only process enabled rules add a rule (17) #16

Merged
merged 10 commits into from
Nov 19, 2024

Conversation

pmpetit
Copy link
Collaborator

@pmpetit pmpetit commented Nov 16, 2024

PR Description

Link to the initial request/ticket

What this PR Provides

  • describe in short sentences the goal of the PR. use lists.

Known limitations / corner cases (if any)

  • if none, use N/A

How to test

  • give what we need to test. example give the sql script to create the context.

Expected results

  • describe the expected result after a run

@pmpetit pmpetit requested a review from GregoireW November 16, 2024 16:04
@pmpetit pmpetit changed the title fix(rules): only process enabled rules fix(rules): only process enabled rules add a rule (17) Nov 16, 2024
@GregoireW GregoireW merged commit 7374c20 into main Nov 19, 2024
6 checks passed
@GregoireW GregoireW deleted the fix/enabled_rules_only branch November 19, 2024 16:45
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.

Param enabled for a rule is not used and all rules are processed all the time.
2 participants