-
Notifications
You must be signed in to change notification settings - Fork 1
Add dependecies.go to install packages
#46
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
Conversation
update syntax
update comment
| # Deploy your app: | ||
| brew install onkernel/tap/kernel | ||
| kernel login # or: export KERNEL_API_KEY=<YOUR_API_KEY> | ||
| kernel deploy index.py |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Bug: Python deploy instruction references wrong entry file
The getNextStepsWithToolInstall function for Python tells users to run kernel deploy index.py, but Python templates use main.py as the entry point (as confirmed by both the test assertions and the actual template files in pkg/templates/python/). Users following these instructions will attempt to deploy a non-existent file.
This PR adds methods to install dependencies after copying templates into a new directory.
Testing
make build && ./bin/kernel createnode_modulesor.venvexistNote
Adds language-specific dependency installation to the create flow with tool availability checks, fallback messaging, and comprehensive tests.
cmd/create.go):create.InstallDependenciesafter copying templates; minor output tweak withpterm.Printfln.pkg/create/dependencies.go):InstallCommandswith spinner.RequiredToolsand print installation/next steps.pkg/create/types.go):RequiredTools,InstallCommands, andTools.CheckToolAvailable.cmd/create_test.go):node_modules,.venv), and next-step output.Written by Cursor Bugbot for commit 66d4e7b. This will update automatically on new commits. Configure here.