Skip to content

fix(js-bazel): guard empty shared runner labels#43

Merged
Jess Sullivan (Jesssullivan) merged 1 commit into
mainfrom
codex/tin1630-shared-runner-label-guard
May 27, 2026
Merged

fix(js-bazel): guard empty shared runner labels#43
Jess Sullivan (Jesssullivan) merged 1 commit into
mainfrom
codex/tin1630-shared-runner-label-guard

Conversation

@Jesssullivan

Copy link
Copy Markdown
Contributor

Summary

  • make runner_mode=shared reject an explicitly empty shared_runner_labels_json
  • document that empty shared labels usually mean a missing caller repo variable
  • add changelog coverage for the guard

This prevents package repos from silently falling back to the default shared tinyland-docker class when vars.PRIMARY_LINUX_RUNNER_LABELS_JSON is unset.

Validation

  • actionlint .github/workflows/js-bazel-package.yml
  • python3 scripts/validate-ci-templates.py internal-refs
  • nix develop --command python3 scripts/validate-ci-templates.py manifest
  • git diff --check

@Jesssullivan Jess Sullivan (Jesssullivan) merged commit ed088f2 into main May 27, 2026
2 checks passed
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