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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
Cli args for wizard #724
base: main
Are you sure you want to change the base?
Cli args for wizard #724
Conversation
@@ -8,10 +8,13 @@ declare global { | |||
namespace LHCI { | |||
namespace WizardCommand { | |||
export interface Options { | |||
wizard?: 'new-project' | 'reset-admin-token'; | |||
wizard?: 'new-project' | 'reset-admin-token' | 'reset-build-token'; |
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.
I think this was missing in the type definition
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.
thanks 馃憤
@patrickhulce Just a heads up :) |
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.
thanks @poteirard!
@@ -489,6 +489,18 @@ The _admin token_ is used to edit or delete data within the project. **KEEP THIS | |||
|
|||
If your lhci server is behind basic auth, you can provide the username and password via the `--basicAuth.password` and `--basicAuth.username` command line flags. | |||
|
|||
All these options are available for `lhci wizard` |
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.
move and link to the configuration docs instead? don't want to have too many places where every flag is documented extensively :)
@@ -8,10 +8,13 @@ declare global { | |||
namespace LHCI { | |||
namespace WizardCommand { | |||
export interface Options { | |||
wizard?: 'new-project' | 'reset-admin-token'; | |||
wizard?: 'new-project' | 'reset-admin-token' | 'reset-build-token'; |
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.
thanks 馃憤
@@ -14,7 +14,7 @@ const btoa = typeof window === 'undefined' ? s => Buffer.from(s).toString('base6 | |||
|
|||
class ApiClient { | |||
/** | |||
* @param {{rootURL: string, fetch?: import('isomorphic-fetch'), URL?: typeof import('url').URL, extraHeaders?: Record<string, string>, basicAuth?: LHCI.ServerCommand.Options['basicAuth']}} options | |||
* @param {{projectExternalUrl?: string, serverBaseUrl?: string, basicAuth?: LHCI.ServerCommand.Options["basicAuth"], wizard?: "new-project" | "reset-admin-token" | "reset-build-token", extraHeaders?: Record<string, string>, storage?: LHCI.ServerCommand.StorageOptions, projectName?: string, projectBaseBranch?: string, rootURL: string}} options |
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.
revert this? it does not accept these options or do anything with them
Hey @moshie, any chance you have time to address Patrick's feedback above? |
Hello!
Thanks for this great repo 馃敟
I just wanted to be able to create a job from my CI to create new projects.
I suppose the easiest way would be to enable CLI args in the CLI
wizard
so I just continued the work of #538Let me know if you need anything else