sentry icon indicating copy to clipboard operation
sentry copied to clipboard

Support more programming languages on GitHub PR Comments

Open bruno-garcia opened this issue 1 year ago • 5 comments

Recently a new feature was released where Sentry can comment on your PRs if there are any unhandled issues being caused by the functions you are modifying in your PR so that you can fix issues while you already have the necessary context.

See: https://sentry.io/changelog/see-unhandled-issues-related-to-open-pull-requests-in-github/

We currently support JavaScript, TypeScript, Python and Go. This covers a lot of our user base already.

https://docs.sentry.io/product/integrations/source-code-mgmt/github/#open-pull-request-comments

This ticket tracks demands for other languages.

Please write a comment with your language of choice to help us prioritize what's coming next.

bruno-garcia avatar Apr 26 '24 22:04 bruno-garcia

I'll go first: I use C# on a few projects that use Sentry. Mainly:

  1. https://github.com/dotnet/nuget-trends
  2. https://github.com/getsentry/symbol-collector

I'd love to help test this feature once C# support is added.

bruno-garcia avatar Apr 26 '24 22:04 bruno-garcia

Routing to @getsentry/product-owners-issues for triage ⏲️

getsantry[bot] avatar Apr 26 '24 22:04 getsantry[bot]

@Dhrumil-Sentry should this go to the ecosystem team or is the limiting factor here something we need to improve in issues?

mrduncan avatar Apr 26 '24 23:04 mrduncan

@mrduncan we need C# codemapping and stack trace linking support before we can do PR Comments

Dhrumil-Sentry avatar Apr 26 '24 23:04 Dhrumil-Sentry

Adding a request for support on Mobile projects - iOS (Cocoa), Android (Java)

gid-sentry avatar Oct 17 '24 18:10 gid-sentry

I have created this milestone for Java: https://github.com/getsentry/sentry/milestone/81 I will be working on it next month.

@bruno-garcia Did you get your request for C#?

armenzg avatar Oct 29 '24 12:10 armenzg

@bruno-garcia not yet :) https://github.com/getsentry/sentry/pull/79641

armenzg avatar Oct 29 '24 12:10 armenzg

I unfortunately don't/can't keep up with GH notifications but glad I found this :)

bruno-garcia avatar Nov 05 '24 03:11 bruno-garcia

C# codemapping is present now(!), going to move this over to ecosystem: https://github.com/getsentry/sentry/pull/79641

mrduncan avatar Nov 06 '24 18:11 mrduncan

Routing to @getsentry/product-owners-settings-integrations for triage ⏲️

getsantry[bot] avatar Nov 06 '24 18:11 getsantry[bot]

+1 for Swift and Kotlin support please

TomSoPolaris avatar Jan 14 '25 16:01 TomSoPolaris

We don't have immediate plans for adding more support but I'll update this ticket when we get to it.

sentaur-athena avatar Jan 15 '25 18:01 sentaur-athena

Customers can manually create their code mappings to enable GH PR comments without having to wait for the automation.

FYI, automatic Java configuration is now complete.

Currently supported languages (The key in that dictionary is the event.platform value): https://github.com/getsentry/sentry/blob/c0bbb9a818d30bb9116cc88b24d3d96df35fc2d3/src/sentry/issues/auto_source_code_config/constants.py#L17-L35

armenzg avatar May 01 '25 20:05 armenzg

We are interested in using with Dart/Flutter

keren-cw avatar Sep 05 '25 20:09 keren-cw

Adding to our backlog.

sentaur-athena avatar Sep 17 '25 23:09 sentaur-athena

Strong interest on our end, we’d love to see first-class Dart/Flutter support

rdurandsaintomer avatar Nov 27 '25 14:11 rdurandsaintomer

We deprecated this feature in favor of upcoming AI PR reviews which includes the same signal. Closing the issue.

sentaur-athena avatar Dec 03 '25 23:12 sentaur-athena