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

refactor: update false positives tuning #174

Merged
merged 2 commits into from
Dec 16, 2024
Merged

Conversation

fzipi
Copy link
Member

@fzipi fzipi commented Dec 14, 2024

what

  • update false positives tuning
  • add quadrant
  • remove reference to old exclusion rules and add plugins

why

  • update docs

@fzipi fzipi requested a review from a team December 14, 2024 13:34
content/concepts/false_positives_tuning.md Outdated Show resolved Hide resolved
content/concepts/false_positives_tuning.md Outdated Show resolved Hide resolved
content/concepts/false_positives_tuning.md Outdated Show resolved Hide resolved
content/concepts/false_positives_tuning.md Outdated Show resolved Hide resolved
content/concepts/false_positives_tuning.md Outdated Show resolved Hide resolved
Copy link

cloudflare-workers-and-pages bot commented Dec 15, 2024

Deploying crs-documentation with  Cloudflare Pages  Cloudflare Pages

Latest commit: 5fcbbfc
Status: ✅  Deploy successful!
Preview URL: https://d296a229.documentation-km5.pages.dev
Branch Preview URL: https://update-false-positives.documentation-km5.pages.dev

View logs

@theseion theseion merged commit 5d6028a into main Dec 16, 2024
2 checks passed
@theseion theseion deleted the update-false-positives branch December 16, 2024 06:11
@dune73
Copy link
Member

dune73 commented Dec 16, 2024

I'm a bit late to the show it seems. The new wording is an improvement. Thank you.

As a note: the preview does not do mermaid it seems.

@fzipi
Copy link
Member Author

fzipi commented Dec 16, 2024

I'll check the mermaid thing.

@theseion
Copy link
Contributor

Mermaid works fine for me in the preview.

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