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 intomainfrom
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 intomainfrom
upstream_as_submodule

Conversation

@jooh
Copy link
Copy Markdown
Owner

@jooh jooh commented Apr 3, 2026

No description provided.

@jooh
Copy link
Copy Markdown
Owner Author

jooh commented Apr 3, 2026

@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