Skip to content

[K9VULN-16477] Add Dart rulesets#936

Merged
jdelgo merged 1 commit into
mainfrom
jdelgo/publish-dart-rules
Jul 9, 2026
Merged

[K9VULN-16477] Add Dart rulesets#936
jdelgo merged 1 commit into
mainfrom
jdelgo/publish-dart-rules

Conversation

@jdelgo

@jdelgo jdelgo commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

What problem are you trying to solve?

Add Dart support for default rules
Jira

@jdelgo
jdelgo marked this pull request as ready for review July 8, 2026 21:19
@jdelgo
jdelgo requested a review from a team as a code owner July 8, 2026 21:19
Copilot AI review requested due to automatic review settings July 8, 2026 21:19
@datadog-prod-us1-3

datadog-prod-us1-3 Bot commented Jul 8, 2026

Copy link
Copy Markdown

Tests

🔄 Datadog auto-retried 1 job - 1 passed on retry View in Datadog

🎯 Code Coverage (details)
Patch Coverage: 100.00%
Overall Coverage: 85.85% (+0.00%)

This comment will be updated automatically if new data arrives.
🔗 Commit SHA: 0e646cb | Docs | Datadog PR Page | Give us feedback!

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Adds Dart to the set of languages for which the CLI fetches “default rulesets” from Datadog, enabling Dart to be included when generating/refreshing the default rules bundle.

Changes:

  • Include "DART" in DEFAULT_RULESETS_LANGUAGES so get_all_default_rulesets() queries default-rulesets/DART as part of the default-ruleset fetch flow.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@jdelgo
jdelgo merged commit 07dd830 into main Jul 9, 2026
91 checks passed
@jdelgo
jdelgo deleted the jdelgo/publish-dart-rules branch July 9, 2026 15:29
@jdelgo jdelgo mentioned this pull request Jul 9, 2026
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