Skip to content

Conversation

@runningcode
Copy link
Contributor

Summary

  • Added build_configuration parameter to sentry_upload_build_filtered calls
  • Matches the same build configuration pattern used for build_app_for_scheme:
    • build_upload_testflight: uses 'Release'
    • build_upload_emerge: uses ENV['CONFIGURATION']

This ensures the Sentry build upload uses the same build configuration as the corresponding app build in each lane.

🤖 Generated with Claude Code

@runningcode runningcode force-pushed the no/add-build-config-to-sentry branch from 7dfeca8 to 1e0dfbf Compare September 18, 2025 17:49
🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
@runningcode runningcode force-pushed the no/add-build-config-to-sentry branch from 1e0dfbf to d1d9363 Compare September 18, 2025 17:51
@emerge-tools
Copy link

emerge-tools bot commented Sep 18, 2025

📸 Snapshot Test

72 unchanged

Name Added Removed Modified Renamed Unchanged Errored Approval
HackerNews
com.emergetools.hackernews.snapshots
0 0 0 0 72 0 N/A

🛸 Powered by Emerge Tools

@trevor-e trevor-e merged commit c7ff1d0 into main Sep 18, 2025
7 checks passed
@trevor-e trevor-e deleted the no/add-build-config-to-sentry branch September 18, 2025 18:16
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