Skip to content

fix(gitutils): Use the GitHub-based package in release-main GitHub Actions#57

Merged
tomgrv merged 5 commits into
developfrom
copilot/fix-nodejs-install-issue
May 23, 2026
Merged

fix(gitutils): Use the GitHub-based package in release-main GitHub Actions#57
tomgrv merged 5 commits into
developfrom
copilot/fix-nodejs-install-issue

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented May 23, 2026

release-main now runs the repository package through GitHub-based resolution with npm settings that let the script execute successfully, while keeping the gitutils installation and verification flow intact.

  • Workflow execution path

    • Use the GitHub-based package spec for release-main instead of npx.
    • Add --legacy-peer-deps so the package install can run install.sh without failing on workspace peer resolution.
    • Keep the same failure handling and alias verification flow.
  • Stub parity

    • Apply the same command change to the gitutils workflow stub so deployed workflow copies stay aligned.
- name: Install GitUtils
  run: npm exec --yes --legacy-peer-deps --package github:tomgrv/devcontainer-features -- devcontainer-features -- gitutils

Copilot AI linked an issue May 23, 2026 that may be closed by this pull request
Copilot AI changed the title [WIP] Fix nodejs install issue to enable release actions Use the local package in release-main GitHub Actions May 23, 2026
Copilot AI requested a review from tomgrv May 23, 2026 18:41
Copilot finished work on behalf of tomgrv May 23, 2026 18:41
Copy link
Copy Markdown
Owner

@tomgrv tomgrv left a comment

Choose a reason for hiding this comment

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

Keep using Github-based package, but make sure it runs smoothly

Copy link
Copy Markdown
Contributor Author

Copilot AI commented May 23, 2026

Keep using Github-based package, but make sure it runs smoothly

Updated in af87950 to use the GitHub-based package again, with an explicit npm exec --package github:tomgrv/devcontainer-features -- devcontainer-features -- gitutils invocation in both workflow copies.

Copilot AI changed the title Use the local package in release-main GitHub Actions Use the GitHub-based package in release-main GitHub Actions May 23, 2026
Copilot finished work on behalf of tomgrv May 23, 2026 19:32
Copilot AI requested a review from tomgrv May 23, 2026 19:32
Copy link
Copy Markdown
Owner

@tomgrv tomgrv left a comment

Choose a reason for hiding this comment

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

Script execution does nothing and exits with error code 1. Looks like install.sh is not called

Copy link
Copy Markdown
Contributor Author

Copilot AI commented May 23, 2026

Script execution does nothing and exits with error code 1. Looks like install.sh is not called

Updated in c1b4386 by adding --legacy-peer-deps so the GitHub-based package install runs install.sh instead of failing during npm workspace peer resolution.

Copilot finished work on behalf of tomgrv May 23, 2026 20:12
Copilot AI requested a review from tomgrv May 23, 2026 20:12
@tomgrv tomgrv marked this pull request as ready for review May 23, 2026 20:15
@tomgrv tomgrv changed the title Use the GitHub-based package in release-main GitHub Actions fix(gitutils): Use the GitHub-based package in release-main GitHub Actions May 23, 2026
@tomgrv tomgrv merged commit 8c46985 into develop May 23, 2026
2 checks passed
@tomgrv tomgrv deleted the copilot/fix-nodejs-install-issue branch May 23, 2026 20:16
@tomgrv tomgrv linked an issue May 24, 2026 that may be closed by this pull request
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.

release action nodejs install issue

2 participants