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

feat(cmd): add next steps hint after template install #417

Merged
merged 4 commits into from
Sep 17, 2024

Conversation

rektdeckard
Copy link
Contributor

  • Reverts to using default terminal color palette, since branded colors do not always play nice with light terminal themes
  • Adds hint text to guide users to start agent after install
Screenshot 2024-09-16 at 5 01 31 PM

@rektdeckard rektdeckard force-pushed the tobias/dex-727-cli-signposting-next-steps branch 3 times, most recently from ad30004 to f504a8f Compare September 16, 2024 23:17
@rektdeckard rektdeckard force-pushed the tobias/dex-727-cli-signposting-next-steps branch 2 times, most recently from f3aba4c to 33e4319 Compare September 17, 2024 01:17
Comment on lines +95 to +101
{
Hidden: true,
Name: "run",
Usage: "Execute a task defined in " + bootstrap.TaskFile,
ArgsUsage: "[TASK] to run in the project's taskfile.yaml",
Action: runTask,
},
Copy link
Contributor Author

@rektdeckard rektdeckard Sep 17, 2024

Choose a reason for hiding this comment

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

@davidzhao hiding this command for now. my other idea was to have a post_create task or smth that runs after we clone, degit, and inflate the environment. All it would do is print the steps to install and get started:

tobiasfried@slate /tmp % lk app sandbox --id 5lfwjtkd6cy
Using default project [Atrium]
Cloning template...
Instantiating environment...
Done! To setup and run your sandbox:

   cd /tmp/foo
   python3 -m venv venv
   source venv/bin/activate
   python3 -m pip install -r requirements.txt
   python3 agent.py start

tobiasfried@slate /tmp %

Copy link
Member

Choose a reason for hiding this comment

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

great, I think this is perfect!

Comment on lines +95 to +101
{
Hidden: true,
Name: "run",
Usage: "Execute a task defined in " + bootstrap.TaskFile,
ArgsUsage: "[TASK] to run in the project's taskfile.yaml",
Action: runTask,
},
Copy link
Member

Choose a reason for hiding this comment

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

great, I think this is perfect!

@rektdeckard rektdeckard merged commit 003645a into main Sep 17, 2024
3 checks passed
@rektdeckard rektdeckard deleted the tobias/dex-727-cli-signposting-next-steps branch September 17, 2024 15: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.

2 participants