search

sentry-demos / ios

iOS Empower Plant
1 stars 4 forks source link

Github Integration - Enable Suspect Commits #23

Closed kpujjigit closed 10 months ago

kpujjigit commented 1 year ago

Problem: Our iOS Demo is not yet configured to showcase suspect commits as part of a demo flow

Goal: To enable suspect commits for the iOS Demo

kpujjigit commented 1 year ago

I will check with product managers on whether or not this is an available feature.

kpujjigit commented 1 year ago

Per Integrations/Ecosystem teams, Stacktrace linking has not been tested for mobile platforms. Restrictions are in-place for mobile SDKs because historically they haven't worked consistently.

ENG expresses interest in revisiting this functionality in the near future, but this will not work as expected in the meantime. Lowering the priority of this tasklist.

kpujjigit commented 11 months ago

Apparently this should work? https://github.com/getsentry/sentry/issues/55044

@kahest wondering if you have any insight? Code Mappings are set up but we havent seen a single suspect commit come in.

kpujjigit commented 10 months ago

This is now enabled by deleting the long-running issue and allowing new events to flow in after making a minor commit to the line where error is thrown. We keep suspect commits for about a year, so the long-standing flagship issue rolled over that period.