getsentry / sentry-cordova

The official Sentry SDK for Cordova
https://sentry.io/
MIT License
68 stars 55 forks source link

Bump sentry-cocoa/ fix issue 311 #316

Closed lucas-zimerman closed 6 months ago

lucas-zimerman commented 8 months ago

This is a possible fix to the issue #311, previously the SDK wasn't being even build due to having an old sentry-cocoa SDK with unsupported minimum version of iOS...

Changes on the SDK

TODO:

lucas-zimerman commented 8 months ago

@brustolin could you check this PR? it got a major bump from the cocoa dependency but I think I got all most of the cases

lucas-zimerman commented 8 months ago

UPDATE: there are still some fixes that will need to be done on Sentry Wizard in order for this PR to fully work

krystofwoldrich commented 7 months ago

@lucas-zimerman Is there a Wizard PR/Issue that we could link here?

lucas-zimerman commented 6 months ago

https://github.com/getsentry/sentry-wizard/pull/481 PR here merged, I'll bump this branch and merge it

github-actions[bot] commented 6 months ago
Fails
:no_entry_sign: Please consider adding a changelog entry for the next release.

Instructions and example for changelog

Please add an entry to CHANGELOG.md to the "Unreleased" section. Make sure the entry includes this PR's number. Example:

## Unreleased

### Fixes/Features

- Bump sentry-cocoa/ fix issue 311 ([#316](https://github.com/getsentry/sentry-cordova/pull/316))

If none of the above apply, you can opt out of this check by adding #skip-changelog to the PR description.

Generated by :no_entry_sign: dangerJS against dbae3609544cd580a10ba8bc2d9236693ccc7a4d