Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ci: NO-JIRA install global nx for codespaces #11

Open
wants to merge 36 commits into
base: staging
Choose a base branch
from

Conversation

lizard-boy
Copy link

ci: install global nx for codespaces

📖 Description

add global nx to codespaces since that's the same way we run our commands locally

Copy link

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

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

PR Summary

This PR modifies the GitHub Codespaces configuration to align it with the local development workflow by installing nx globally and updating the development environment setup.

  • Added .devcontainer/install.sh to install pnpm and nx globally in Codespaces
  • Modified .devcontainer/devcontainer.json to use the new install script and set bash as the default shell
  • Removed pnpm feature and replaced updateContentCommand with postCreateCommand in devcontainer.json
  • Updated Vitest extension in the devcontainer configuration

2 file(s) reviewed, 2 comment(s)
Edit PR Review Bot Settings

Comment on lines +3 to +5
pnpm bin -g
source /home/node/.bashrc
pnpm bin -g
Copy link

Choose a reason for hiding this comment

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

style: Redundant pnpm bin -g calls, remove one

Comment on lines +1 to +6
#!/bin/bash
curl -fsSL https://get.pnpm.io/install.sh | sh -
pnpm bin -g
source /home/node/.bashrc
pnpm bin -g
pnpm -g add nx
Copy link

Choose a reason for hiding this comment

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

style: Add error handling and logging to improve script robustness

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