Skip to content

Use a snapshot-env-free macOS executor for update-error-codes#6995

Merged
AlvaroBrey merged 1 commit into
mainfrom
fix-error-codes-macos-executor
Jun 15, 2026
Merged

Use a snapshot-env-free macOS executor for update-error-codes#6995
AlvaroBrey merged 1 commit into
mainfrom
fix-error-codes-macos-executor

Conversation

@AlvaroBrey

@AlvaroBrey AlvaroBrey commented Jun 15, 2026

Copy link
Copy Markdown
Contributor

The default macos-executor has two pipeline parameters for snapshots as environment variables. When passed to an external orb, it can't resolve them, causing an error

To work around this, I've defined a separate executor for external use that just hardcodes the envs to false.


Note

Low Risk
CI configuration only; no application or SDK code changes, and snapshot env behavior for the error-codes job is intentionally fixed to false.

Overview
Fixes CircleCI Unknown variable(s) failures when the revenuecat/update-error-codes orb job receives the default macos-executor, whose environment still references << pipeline.parameters.generate_snapshots >> and related snapshot flags that do not resolve inside the orb’s config scope.

Adds macos-executor-external, mirroring the standard macOS executor but hardcoding CIRCLECI_TESTS_GENERATE_SNAPSHOTS and CIRCLECI_TESTS_GENERATE_REVENUECAT_UI_SNAPSHOTS to "false", and wires the update-error-codes workflow to use it instead of macos-executor.

Reviewed by Cursor Bugbot for commit 1e846fc. Bugbot is set up for automated code reviews on this repo. Configure here.

@AlvaroBrey AlvaroBrey self-assigned this Jun 15, 2026
macos-executor embeds pipeline.parameters.generate_snapshots / generate_revenuecatui_snapshots
in its environment, which don't resolve when the executor is evaluated outside this config's
pipeline-parameter scope (e.g. inside the revenuecat/update-error-codes orb job — the #13 merge
errored the iOS pipeline with "Unknown variable(s)"). Add macos-executor-external (macos-executor
minus that snapshot env) and use it for the update-error-codes job.
@AlvaroBrey AlvaroBrey force-pushed the fix-error-codes-macos-executor branch from 83977c4 to 1e846fc Compare June 15, 2026 08:13
@AlvaroBrey AlvaroBrey requested a review from a team June 15, 2026 08:17
@AlvaroBrey AlvaroBrey marked this pull request as ready for review June 15, 2026 08:17
@AlvaroBrey AlvaroBrey requested a review from a team as a code owner June 15, 2026 08:17
@AlvaroBrey AlvaroBrey removed the request for review from a team June 15, 2026 08:17

@ajpallares ajpallares left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks for the fix! I think it makes sense!

@AlvaroBrey AlvaroBrey merged commit 4a533d2 into main Jun 15, 2026
18 of 20 checks passed
@AlvaroBrey AlvaroBrey deleted the fix-error-codes-macos-executor branch June 15, 2026 08:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants