Skip to content

new approach - #188

Open
fulldecent wants to merge 3 commits into
mainfrom
bump-deps
Open

new approach#188
fulldecent wants to merge 3 commits into
mainfrom
bump-deps

Conversation

@fulldecent

Copy link
Copy Markdown
Owner

No description provided.

Copilot AI review requested due to automatic review settings July 30, 2026 17:41

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Updates the Dev Container provisioning command to use a new chained-command approach for installing Ruby/Node dependencies and bootstrapping Corepack/Yarn, aiming to simplify container setup for local development.

Changes:

  • Replaces the Dev Container postCreateCommand with a &&-chained command sequence.
  • Switches Corepack setup from corepack prepare yarn@4.18.0 --activate to corepack install.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread .devcontainer/devcontainer.json Outdated
},
// Ruby and Node versions come from .ruby-version and .nvmrc (single source of truth, shared with CI)
"postCreateCommand": "/bin/bash -c 'set -e; rbenv install --skip-existing; rbenv rehash; source \"$NVM_DIR/nvm.sh\"; nvm install; nvm use; npm install -g corepack; corepack enable; corepack prepare yarn@4.18.0 --activate; yarn install --immutable; bundle install'",
"postCreateCommand": "rbenv install --skip-existing && rbenv rehash && . \"$NVM_DIR/nvm.sh\" && nvm install && nvm use && npm install -g corepack && corepack enable && corepack install && yarn install --immutable && bundle install",
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.

2 participants