Skip to content

Change IDE crate code owners to be the full team#888

Merged
abrooksv merged 1 commit intomainfrom
changeIdeCodeOwners
Apr 22, 2026
Merged

Change IDE crate code owners to be the full team#888
abrooksv merged 1 commit intomainfrom
changeIdeCodeOwners

Conversation

@abrooksv
Copy link
Copy Markdown
Contributor

What problem are you trying to solve?

Just changing the code owners for the IDE code to be the full team

Copilot AI review requested due to automatic review settings April 21, 2026 21:52
@abrooksv abrooksv requested a review from a team as a code owner April 21, 2026 21:52
@abrooksv abrooksv requested a review from jacobotb April 21, 2026 21:52
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates repository ownership rules so the IDE server code path is owned by the full IDE integration team, aligning review responsibility with the intended group.

Changes:

  • Changed CODEOWNERS for /crates/bins/src/bin/datadog_static_analyzer_server/ide/**/* from @DataDog/ide-integration-static-analysis-server to @DataDog/ide-integration.

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

@datadog-datadog-prod-us1
Copy link
Copy Markdown

🎯 Code Coverage (details)
Patch Coverage: -1.00%
Overall Coverage: 85.01% (+0.00%)

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

Copy link
Copy Markdown
Contributor

@robertohuertasm-datadog robertohuertasm-datadog left a comment

Choose a reason for hiding this comment

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

LGTM! 🚀

@abrooksv abrooksv merged commit d4086c1 into main Apr 22, 2026
94 checks passed
@abrooksv abrooksv deleted the changeIdeCodeOwners branch April 22, 2026 15:53
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.

4 participants