Skip to content

Conversation

@antonis
Copy link
Contributor

@antonis antonis commented Sep 26, 2025

📢 Type of change

  • Bugfix
  • New feature
  • Enhancement
  • Refactoring

📜 Description

Temporarily deactivate swiftlint since it is blocking the release process

Opened an issue to revisit this #5213

💡 Motivation and Context

Check #5206 (comment)

💚 How did you test it?

CI

📝 Checklist

  • I added tests to verify changes
  • No new PII added or SDK only sends newly added PII if sendDefaultPII is enabled
  • I updated the docs if needed.
  • I updated the wizard if needed.
  • All tests passing
  • No breaking changes

🔮 Next steps

#skip-changelog

Copy link
Collaborator

@lucas-zimerman lucas-zimerman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM if build passes

@antonis
Copy link
Contributor Author

antonis commented Sep 26, 2025

Merging as the Lint job is 🟢

@antonis antonis merged commit 3a89136 into release/7.2.0 Sep 26, 2025
62 of 64 checks passed
@antonis antonis deleted the antonis/temporarily-deactivate-swift-lint branch September 26, 2025 15:27
@github-actions
Copy link
Contributor

iOS (legacy) Performance metrics 🚀

  Plain With Sentry Diff
Startup time 1232.00 ms 1238.42 ms 6.42 ms
Size 2.63 MiB 3.98 MiB 1.34 MiB

@github-actions
Copy link
Contributor

Android (new) Performance metrics 🚀

  Plain With Sentry Diff
Startup time 360.65 ms 425.31 ms 64.66 ms
Size 7.15 MiB 8.41 MiB 1.26 MiB

@github-actions
Copy link
Contributor

iOS (new) Performance metrics 🚀

  Plain With Sentry Diff
Startup time 1224.94 ms 1228.37 ms 3.43 ms
Size 3.19 MiB 4.54 MiB 1.36 MiB

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants