Skip to content

Merge pull request #21 from jooh/codex/fix-build-error-for-macos-13-us-default ci: remove unsupported macOS runner from install-script matrix#22

Closed
jooh wants to merge 3 commits into
mainfrom
upstream_as_submodule
Closed

Merge pull request #21 from jooh/codex/fix-build-error-for-macos-13-us-default ci: remove unsupported macOS runner from install-script matrix#22
jooh wants to merge 3 commits into
mainfrom
upstream_as_submodule

Conversation

@jooh

@jooh jooh commented Apr 3, 2026

Copy link
Copy Markdown
Owner

No description provided.

@jooh

jooh commented Apr 3, 2026

Copy link
Copy Markdown
Owner Author

@codex solve merge conflicts

@chatgpt-codex-connector

Copy link
Copy Markdown

Summary

  • Updated the install-script workflow matrix to use matrix.include with a single Ubuntu runner entry, preserving Ubuntu-only execution while making this section less conflict-prone for future merges. .github/workflows/dev-containers.ymlL152-L169
  • Added an explicit comment that this structure is intended to reduce branch matrix drift/merge conflicts while keeping the existing macOS availability rationale. .github/workflows/dev-containers.ymlL159-L162
  • Committed the change on branch work (6f3604d) and created a follow-up PR via the make_pr tool titled “ci: resolve install-script merge conflicts by normalizing runner matrix”.

Testing

  • node -e "const fs=require('fs'); const yaml=require('js-yaml'); yaml.load(fs.readFileSync('.github/workflows/dev-containers.yml','utf8')); console.log('YAML OK');"

View task →

@jooh jooh closed this Apr 3, 2026
@jooh jooh deleted the upstream_as_submodule branch April 3, 2026 19:40
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