Skip to content

Conversation

@suejung-sentry
Copy link
Member

@suejung-sentry suejung-sentry commented Dec 5, 2025

Use new feature flag to gate rollout of on_new_commit=True so we run code review on every commit (instead of just when PR opened).
We'll roll this out to batches of orgs at a time

PREVENT-472

@github-actions github-actions bot added the Scope: Backend Automatically applied to PRs that change backend components label Dec 5, 2025
@codecov
Copy link

codecov bot commented Dec 5, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@             Coverage Diff             @@
##           master   #104489      +/-   ##
===========================================
- Coverage   80.53%    80.53%   -0.01%     
===========================================
  Files        9350      9350              
  Lines      400099    400108       +9     
  Branches    25660     25660              
===========================================
+ Hits       322216    322221       +5     
- Misses      77415     77419       +4     
  Partials      468       468              

@suejung-sentry suejung-sentry marked this pull request as ready for review December 5, 2025 23:23
@suejung-sentry suejung-sentry requested a review from a team as a code owner December 5, 2025 23:23
from sentry.models.repository import Repository
from sentry.prevent.models import PreventAIConfiguration
from sentry.prevent.types.config import PREVENT_AI_CONFIG_DEFAULT
from sentry.prevent.types.config import PREVENT_AI_CONFIG_DEFAULT, PREVENT_AI_CONFIG_DEFAULT_V1
Copy link
Contributor

Choose a reason for hiding this comment

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

at some point we should probably rename "prevent" to "code review"
just noting it down tho

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Scope: Backend Automatically applied to PRs that change backend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants