Skip to content

Updates to EAS build hooks#5773

Merged
alwx merged 1 commit intomainfrom
alwx/bugfix/eas-build-hook-smallish-fix
Mar 5, 2026
Merged

Updates to EAS build hooks#5773
alwx merged 1 commit intomainfrom
alwx/bugfix/eas-build-hook-smallish-fix

Conversation

@alwx
Copy link
Contributor

@alwx alwx commented Mar 5, 2026

📢 Type of change

  • Bugfix
  • New feature
  • Enhancement
  • Refactoring

📜 Description

Two small fixes to EAS build hooks:

  • When a user explicitly adds up sentry-eas-build-on-success, it now does the right thing without needing an extra env var
  • Dynamic version of SDK gets attached to sentry.javascript.react-native.eas-build-hooks SDK name

📝 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

#skip-changelog

- When a user explicitly adds up sentry-eas-build-on-success, it now does the right thing without needing an extra env var
- Dynamic version of SDK gets attached to `sentry.javascript.react-native.eas-build-hooks` SDK name
@alwx alwx force-pushed the alwx/bugfix/eas-build-hook-smallish-fix branch from 80739df to accdfd6 Compare March 5, 2026 11:07
@github-actions
Copy link
Contributor

github-actions bot commented Mar 5, 2026

Semver Impact of This PR

None (no version bump detected)

📋 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.

@alwx alwx added the ready-to-merge Triggers the full CI test suite label Mar 5, 2026
@alwx alwx self-assigned this Mar 5, 2026
Copy link
Contributor

@antonis antonis left a comment

Choose a reason for hiding this comment

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

LGTM 🚀 Thank you for iterating on the feedback 🙇

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!

@alwx alwx enabled auto-merge (squash) March 5, 2026 11:33
@alwx alwx merged commit 752e659 into main Mar 5, 2026
69 of 113 checks passed
@alwx alwx deleted the alwx/bugfix/eas-build-hook-smallish-fix branch March 5, 2026 11:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready-to-merge Triggers the full CI test suite skip-changelog

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants