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

[Issue #40] Fix CLI dependencies #80

Merged
merged 4 commits into from
Feb 12, 2025
Merged

Conversation

widal001
Copy link
Collaborator

Summary

Fixes how the path tsp CLI is resolved in the CommonGrants CLI, so that cg init and cg compile can serve as thin-wrappers for tsp init and tsp compile

Changes proposed

What was added, updated, or removed in this PR.

  • Adds @typespec/compiler as a direct dependency
  • Changes how we resolve the path to the tsp binary
  • Adds back the default-api template

Context for reviewers

Testing instructions, background context, more in-depth details of the implementation, and anything else you'd like to call out or ask reviewers. Explain how the changes were verified.

  1. Checkout the PR locally
  2. Change directory into the cli/: cd cli
  3. Run npm pack
  4. In a separate clean directory install the tarball: npm install <path-to-tarball>
  5. Run npx cg init

@widal001 widal001 merged commit cacef85 into main Feb 12, 2025
2 checks passed
@widal001 widal001 deleted the issue-40-fix-cli-dependencies branch February 12, 2025 14:58
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.

1 participant