Skip to content
This repository was archived by the owner on Sep 11, 2025. It is now read-only.

Conversation

@mattjohnsonpint
Copy link
Contributor

Adds a --no-runtime flag to modus sdk install, and uses it when installing SDKs during modus build. This is primarily to allow modus build to be used in CI/CD without an unused Modus Runtime becoming installed into the build runner.

Also updates CLI dependencies to latest.

@mattjohnsonpint mattjohnsonpint requested review from a team and Copilot September 3, 2025 17:38
Copy link
Contributor

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 adds a --no-runtime flag to the modus sdk install command to skip runtime installation during CI/CD builds, and updates CLI dependencies to their latest versions.

  • Introduces --no-runtime flag to SDK install command for conditional runtime installation
  • Updates build command to use the new flag when installing SDKs during builds
  • Updates multiple CLI dependencies to their latest versions

Reviewed Changes

Copilot reviewed 4 out of 5 changed files in this pull request and generated no comments.

File Description
cli/src/commands/sdk/install/index.ts Adds --no-runtime flag and conditional runtime installation logic
cli/src/commands/build/index.ts Updates SDK installation call to include --no-runtime flag
cli/package.json Updates CLI dependencies to latest versions
CHANGELOG.md Documents the new feature in changelog
Files not reviewed (1)
  • cli/package-lock.json: Language not supported

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@mattjohnsonpint mattjohnsonpint merged commit c8f9ce6 into main Sep 3, 2025
7 checks passed
@mattjohnsonpint mattjohnsonpint deleted the mjp/cli branch September 3, 2025 17:41
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants