Skip to content
Open
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
9 changes: 9 additions & 0 deletions .github/CODEOWNERS
Original file line number Diff line number Diff line change
Expand Up @@ -683,3 +683,12 @@ pnpm-lock.yaml @getsentry/owners-js-de
/static/CLAUDE.md @getsentry/design-engineering
/scripts/analyze-styled.ts @getsentry/design-engineering
## End of Frontend Platform

# Prevent
/src/sentry/prevent/ @getsentry/prevent
/src/sentry/overwatch/ @getsentry/prevent
/src/sentry/overwatch_webhooks/ @getsentry/prevent
/static/app/components/prevent/ @getsentry/prevent
/static/app/views/prevent/ @getsentry/prevent
/static/app/views/nav/secondary/sections/prevent/ @getsentry/prevent
# End of Prevent
Comment on lines +687 to +694

Choose a reason for hiding this comment

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

[CompanyBestPractice]

The section comment inconsistency could cause confusion when navigating the CODEOWNERS file. The opening section uses # Prevent while the closing section uses # End of Prevent, but other sections in the file use ## Prevent and ## End of [Section Name] format (e.g., ## Crons and ## End Crons). Consider updating for consistency:

Suggested Change
Suggested change
# Prevent
/src/sentry/prevent/ @getsentry/prevent
/src/sentry/overwatch/ @getsentry/prevent
/src/sentry/overwatch_webhooks/ @getsentry/prevent
/static/app/components/prevent/ @getsentry/prevent
/static/app/views/prevent/ @getsentry/prevent
/static/app/views/nav/secondary/sections/prevent/ @getsentry/prevent
# End of Prevent
## Prevent
/src/sentry/prevent/ @getsentry/prevent
/src/sentry/overwatch/ @getsentry/prevent
/src/sentry/overwatch_webhooks/ @getsentry/prevent
/static/app/components/prevent/ @getsentry/prevent
/static/app/views/prevent/ @getsentry/prevent
/static/app/views/nav/secondary/sections/prevent/ @getsentry/prevent
## End of Prevent

Committable suggestion

Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation.

Context for Agents
[**CompanyBestPractice**]

The section comment inconsistency could cause confusion when navigating the CODEOWNERS file. The opening section uses `# Prevent` while the closing section uses `# End of Prevent`, but other sections in the file use `## Prevent` and `## End of [Section Name]` format (e.g., `## Crons` and `## End Crons`). Consider updating for consistency:

<details>
<summary>Suggested Change</summary>

```suggestion
## Prevent
/src/sentry/prevent/                                      @getsentry/prevent
/src/sentry/overwatch/                                    @getsentry/prevent
/src/sentry/overwatch_webhooks/                           @getsentry/prevent
/static/app/components/prevent/                           @getsentry/prevent
/static/app/views/prevent/                                @getsentry/prevent
/static/app/views/nav/secondary/sections/prevent/         @getsentry/prevent
## End of Prevent
```

⚡ **Committable suggestion**

Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation.

</details>

File: .github/CODEOWNERS
Line: 694