Skip to content

Clarify environment-scoped config and validate missing settings#189

Merged
elasticdog merged 2 commits intomainfrom
push-qmrnoptwxukv
Feb 12, 2026
Merged

Clarify environment-scoped config and validate missing settings#189
elasticdog merged 2 commits intomainfrom
push-qmrnoptwxukv

Conversation

@elasticdog
Copy link
Member

No description provided.

This was only working coincidentally for our signing setup, the values
weren't actually being passed through from the caller repository.
@github-actions github-actions bot added documentation Improvements or additions to documentation github_actions labels Feb 12, 2026
@elasticdog elasticdog added this pull request to the merge queue Feb 12, 2026
Merged via the queue into main with commit 9a90b9e Feb 12, 2026
7 checks passed
@elasticdog elasticdog deleted the push-qmrnoptwxukv branch February 12, 2026 18:51
elasticdog added a commit to EarthmanMuons/whatchord that referenced this pull request Feb 12, 2026
The signing was only working previously by coincidence, since we
moved these values into a repository environment, they aren't actually
available here in the call site, only inside of the jobs defined in
the reusable workflows, so we rely on convention, documentation, and
validation on the other side to ensure we've done this correctly for
both the signing and release environmental jobs.

See:
- EarthmanMuons/reusable-workflows#189
github-merge-queue bot pushed a commit to EarthmanMuons/whatchord that referenced this pull request Feb 12, 2026
The signing was only working previously by coincidence, since we
moved these values into a repository environment, they aren't actually
available here in the call site, only inside of the jobs defined in
the reusable workflows, so we rely on convention, documentation, and
validation on the other side to ensure we've done this correctly for
both the signing and release environmental jobs.

See:
- EarthmanMuons/reusable-workflows#189
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation github_actions

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant