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

Added confirmation for opengb init command #279

Merged
merged 3 commits into from
May 25, 2024
Merged

Conversation

vr-varad
Copy link
Contributor

@vr-varad vr-varad commented May 18, 2024

Implementation

Say which files were created
Link to docs on getting started

solution Implementation

The templateProject function now returns a Promise of type string[], tracking the paths of created files and directories and resolving with this array. Error handling is implemented to catch and log errors, returning an empty array if creation fails.

Resolves #277

@vr-varad
Copy link
Contributor Author

Hey @NathanFlurry I have implemented the solution as discussed. I would appreciate it if you could review it at your earliest convenience. Please let me know if you have any questions or require further clarification. Thank you.

Copy link
Member

@NathanFlurry NathanFlurry left a comment

Choose a reason for hiding this comment

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

welcome, thanks for the pr!

using this in practice, i don't think it actually makes sense to print the files. would simplify intro to something like:

Project created successfully

Created config file at backend.yaml

Get started at https://opengb.dev/introduction

src/cli/commands/init.ts Outdated Show resolved Hide resolved
@vr-varad
Copy link
Contributor Author

@NathanFlurry I have made the required changes.

@NathanFlurry NathanFlurry merged commit 43fe05d into rivet-gg:main May 25, 2024
@NathanFlurry
Copy link
Member

@vr-varad thanks for the pr! apologies for the ticket being wrong, i pushed some small tweaks.

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.

Add confirmation for opengb init command
2 participants