-
Notifications
You must be signed in to change notification settings - Fork 93
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
[ENH] - Improve QHub Setup experience #1372
Comments
I am a bit confused by this - can you elaborate @aktech? |
A good example of init script + CLI https://docs.coiled.io/user_guide/getting_started.html
|
The idea was to have a set of questions in the qhub init cli with reasonable defaults, for e.g. $ qhub init
Welcome to QHub Initialisation
Q. What is the proiect name (my-awesome project)?
qhub-deployment-aws
Q. What domain would you like to host QHub at (e.g. myqhub.example.com)?
.... This is just a suggestion. This issue is more like a RFP, we can then decide the final API from the potential ideas based on the user friendliness. |
@asmijafar20 and I have spent sometime mapping out the current state of the CLI and the associated function calls. This concludes the "exploration" phase of this project. For more details, see the following links:
The next steps are for @asmijafar20 to work on an MVP using Typer (in a separate repo) and then for use to integrate those changes in a branch here on qhub. This will ensure that these changes work with the existing codebase and CI. |
@asmijafar20 and I have been working on the new typer CLI and have a decent framework on which to build. For more details see this branch. |
Feature description
This is an umbrella issue for improving QHub's setup experience. This includes:
qhub init
call - could cover [ENH] - Add--dry-run
option to qhub installation #1368qhub init
could have the user answer a list of questions (instead of passing all those flags)qhub init
#1267): an example of this could be validating that theproject_name
is in the correct format for cloud providers (like say If using AWS: names should not start with the string "aws”)Value and/or benefit
Easier to onboard new users, easier for overall testing as well.
Anything else?
No response
The text was updated successfully, but these errors were encountered: