Skip to content

ci: fix devcontainer-collection.json by publishing features together#41

Merged
mailaenderli merged 3 commits intomainfrom
fix/fix-collection-metadata
Apr 21, 2026
Merged

ci: fix devcontainer-collection.json by publishing features together#41
mailaenderli merged 3 commits intomainfrom
fix/fix-collection-metadata

Conversation

@mailaenderli
Copy link
Copy Markdown
Contributor

No description provided.

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR adjusts the CI/release flow so that all devcontainer features are published in a single operation, preventing the devcontainer collection metadata from being overwritten/incorrect when publishing feature-by-feature.

Changes:

  • Replace per-feature publish tasks with a single Features:Publish task that prepares and publishes all features together.
  • Split CI into a build/test matrix job and a gated publish job that runs only on main.
  • Add a reusable composite GitHub Action to standardize Go/Node/DevContainer CLI setup across jobs.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.

File Description
build/build.go Introduces publishFeatures() and a Features:Publish task to publish all features together.
.github/workflows/ci.yml Refactors CI to use a shared setup action and adds a dedicated publish job.
.github/actions/setup-env/action.yml New composite action to install Go, Node, and @devcontainers/cli.
.devcontainer/devcontainer.json Updates the devcontainer Go feature version used for local development.

Comment thread build/build.go
Copy link
Copy Markdown
Member

@Roemer Roemer left a comment

Choose a reason for hiding this comment

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

LGTM

@mailaenderli mailaenderli merged commit ebee6aa into main Apr 21, 2026
29 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.

3 participants