mehulmathur001

Results 9 comments of mehulmathur001

Hi @weihanglo @jarhodes314 are there any other alternatives for getting this to work ?

Hi @SlySven, correct. But this is related to this issue : https://github.com/getsentry/sentry-native/issues/838. So I was just trying if it works in lower versions as specified in above issue. I will...

Hi @vadi2 @SlySven please review this PR.

Hi @Kebap, this PR can be tested by adding sentry env. variables. Currently, I'm intentionally making macOS build to fail, for checking runtime error is getting logged on sentry.

@SlySven @vadi2 Please have a look at the below test runtime error that I had thrown for testing sentry. https://github.com/Mudlet/Mudlet/assets/160306774/ede2b997-d9b8-40a0-8d95-707eaeea6257

@SlySven @vadi2 @Kebap can you please review this PR ?

> I think this needs to have a conditional for inclusion as not everyone building on MacOS will want their crashes to be reported back to Mudlet HQ. Yes @SlySven....

> Apart from that last tiny thing this looks good to me. However I cannot test this myself as it needs someone with a MacOS system to test it for...

Hi @SlySven, I'll copy the checks like we have for macOS here too. Currently I'm just trying to make the windows build pass somehow.