Skip to content

[DO NOT MERGE] Test snapshots only under a new bundle ID - #842

Draft
NicoHinderling wants to merge 3 commits into
mainfrom
codex/test-snapshot-ci-2026-09-16
Draft

NicoHinderling wants to merge 3 commits into
mainfrom
codex/test-snapshot-ci-2026-09-16

Conversation

@NicoHinderling

@NicoHinderling NicoHinderling commented Sep 16, 2026

Copy link
Copy Markdown
Contributor

Purpose

Temporary snapshot CI test using a brand-new bundle/app ID. Do not merge.

Only upload for this PR

Reuse the existing iOS SnapshotPreviews workflow and its complete aggregated image set, uploading one snapshot build to Sentry project hackernews-ios under:

com.emergetools.hackernews.snapshot-ci-pr842-3f62449f58a3

This overrides the snapshot upload's app_id; application signing and installed bundle identifiers are unchanged. The synthetic orange story-title UI change remains.

Skipped for PR #842

  • Android snapshots and iOS Swift Snapshot Testing uploads.
  • Android and iOS size-analysis builds/uploads.
  • iOS PR build-distribution builds/uploads.

Only the iOS SnapshotPreviews build/generate/upload pipeline remains active. GitHub artifacts still transfer build products and images between those jobs. The exclusions and ID override apply specifically to PR #842; other PRs and main keep their existing behavior.

The original revision already completed normal uploads. This update isolates subsequent runs; it does not delete historical uploads.

Validation

  • Passed diff checks, workflow YAML parsing, and Fastfile Ruby syntax validation using locally installed Ruby 3.3.0.
  • Exercised workflow conditions and app ID selection for this PR, another PR, and an absent PR number.
  • Exercised the upload lane with a stubbed action: the custom/default app IDs work and other upload parameters remain unchanged.
  • Full builds/uploads are left to CI; the exact local Ruby version and locked gems are not installed.

Close after testing; not intended for release.

@sentry

sentry Bot commented Sep 16, 2026

Copy link
Copy Markdown

📲 Install Builds

iOS

🔗 App Name App ID Version Configuration
HackerNews com.emergetools.hackernews 3.10 (1) Release
HackerNews com.emergetools.hackernews 3.10 (1) AdHoc

Android

🔗 App Name App ID Version Configuration
Hacker News com.emergetools.hackernews 1.0.6 (17) release

⚙️ hackernews-android Build Distribution Settings

@sentry

sentry Bot commented Sep 16, 2026

Copy link
Copy Markdown

Sentry Snapshot Testing

Name Added Removed Changed Renamed Unchanged Skipped Status
com.emergetools.hackernews.snapshot-ci-pr842-3f62449f58a3
com.emergetools.hackernews.snapshot-ci-pr842-3f62449f58a3
- - - - - - ✅ 132 uploaded

Base commit 6eea866e77e956a32fe8435b7427501284f47517 did not produce snapshots to compare against. Did its snapshot job fail? Try rebasing this branch on a commit with a successful snapshot job.

⚙️ hackernews-ios Snapshot Settings

@sentry

sentry Bot commented Sep 16, 2026

Copy link
Copy Markdown

Size Analysis

3 components analyzed

iOS Builds

Name Configuration Version Download Size Install Size
HackerNews (iOS)
com.emergetools.hackernews
Release 3.10 (1) 4.4 MB (-2.3 MB) 7.6 MB (-2.4 MB)
HackerNews (iOS)
com.emergetools.hackernews
AdHoc 3.10 (1) 6.7 MB (+222 B) 10.1 MB (0 B)

Android Builds

Name Configuration Version Download Size Uncompressed Size
Hacker News (Android)
com.emergetools.hackernews
release 1.0.6 (17) 3.0 MB (-121 B) 6.9 MB (-43 B)

Configure hackernews-android status check rules

@NicoHinderling NicoHinderling changed the title [DO NOT MERGE] Test snapshot CI with synthetic UI changes [DO NOT MERGE] Test snapshots only under a new bundle ID Sep 16, 2026
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.

1 participant