Skip to content

feat(issues): infer project platform if not set #95402

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

Open
wants to merge 8 commits into
base: master
Choose a base branch
from

Conversation

cvxluo
Copy link

@cvxluo cvxluo commented Jul 13, 2025

If a project ends up with no platform (e.g. by API creation), we attempt to infer it from its events. We go by the following:

  1. if a project has a non-null platform that we did not previously infer, do not do anything
  2. if a project has no platform, set platform to the next event platform it receives
  3. if a project has a platform that we've previously inferred and it receives an event that does not match that platform, set the project platform to other

This creates a new project option sentry:project_platform_inferred, which is set to the inferred platform.

Related:

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

codecov bot commented Jul 13, 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   #95402      +/-   ##
==========================================
- Coverage   87.84%   87.80%   -0.04%     
==========================================
  Files       10502    10490      -12     
  Lines      608126   606000    -2126     
  Branches    23719    23710       -9     
==========================================
- Hits       534228   532124    -2104     
+ Misses      73540    73518      -22     
  Partials      358      358              

@cvxluo cvxluo marked this pull request as ready for review July 14, 2025 02:19
@cvxluo cvxluo requested a review from a team as a code owner July 14, 2025 02:19
cursor[bot]

This comment was marked as outdated.

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.

2 participants