Skip to content

chore: Remove App Store Connect credentials from CI workflows where not necessary#7447

Merged
itaybre merged 1 commit intomainfrom
itay/remove_unnecesary_env_variables
Feb 13, 2026
Merged

chore: Remove App Store Connect credentials from CI workflows where not necessary#7447
itaybre merged 1 commit intomainfrom
itay/remove_unnecesary_env_variables

Conversation

@itaybre
Copy link
Contributor

@itaybre itaybre commented Feb 12, 2026

📜 Description

Removes AppStore credentials from workflows which don't need it

#skip-changelog

💡 Motivation and Context

These workflows are using Match, so we don't need AppStore credentials

💚 How did you test it?

📝 Checklist

You have to check all boxes before merging:

  • I added tests to verify the 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.
  • Review from the native team if needed.
  • No breaking change or entry added to the changelog.
  • No breaking change for hybrid SDKs or communicated to hybrid SDKs.

Closes #7448

@itaybre itaybre added the ready-to-merge Use this label to trigger all PR workflows label Feb 12, 2026
@github-actions
Copy link
Contributor

github-actions bot commented Feb 12, 2026

Semver Impact of This PR

🟢 Patch (bug fixes)

📋 Changelog Preview

This is how your changes will appear in the changelog.
Entries from this PR are highlighted with a left border (blockquote style).


This PR will not appear in the changelog.


🤖 This preview updates automatically when you update the PR.

Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.

@github-actions
Copy link
Contributor

github-actions bot commented Feb 12, 2026

Performance metrics 🚀

  Plain With Sentry Diff
Startup time 1230.87 ms 1259.26 ms 28.38 ms
Size 24.14 KiB 1.11 MiB 1.09 MiB

Baseline results on branch: main

Startup times

Revision Plain With Sentry Diff
9a0a3cd 1220.85 ms 1243.69 ms 22.83 ms
50e7b3e 1221.54 ms 1250.81 ms 29.27 ms
1b74fd4 1219.43 ms 1250.17 ms 30.73 ms
58a9225 1211.40 ms 1238.88 ms 27.48 ms
a7d7fb6 1209.53 ms 1235.91 ms 26.38 ms
1d0feed 1213.31 ms 1247.44 ms 34.14 ms
02391fd 1205.51 ms 1228.66 ms 23.15 ms
3bff9ff 1217.72 ms 1246.43 ms 28.71 ms
17b7e88 1223.04 ms 1253.48 ms 30.43 ms
79e2bb8 1216.37 ms 1242.42 ms 26.05 ms

App size

Revision Plain With Sentry Diff
9a0a3cd 24.14 KiB 1.08 MiB 1.06 MiB
50e7b3e 24.14 KiB 1.04 MiB 1.02 MiB
1b74fd4 24.14 KiB 1.10 MiB 1.08 MiB
58a9225 24.14 KiB 1.06 MiB 1.04 MiB
a7d7fb6 24.14 KiB 1.06 MiB 1.04 MiB
1d0feed 24.14 KiB 1.05 MiB 1.03 MiB
02391fd 24.14 KiB 1.11 MiB 1.09 MiB
3bff9ff 24.14 KiB 1.11 MiB 1.09 MiB
17b7e88 24.14 KiB 1.06 MiB 1.04 MiB
79e2bb8 24.14 KiB 1.04 MiB 1.02 MiB

Copy link
Member

@philprime philprime left a comment

Choose a reason for hiding this comment

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

LGTM, we can always add it back in if needed

@itaybre itaybre enabled auto-merge (squash) February 13, 2026 17:57
@itaybre itaybre merged commit daf8b80 into main Feb 13, 2026
224 of 275 checks passed
@itaybre itaybre deleted the itay/remove_unnecesary_env_variables branch February 13, 2026 17:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready-to-merge Use this label to trigger all PR workflows

Projects

None yet

Development

Successfully merging this pull request may close these issues.

chore: Remove App Store Connect credentials from CI workflows where not necessary

2 participants