-
Notifications
You must be signed in to change notification settings - Fork 5
Cli doc #117
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
louisdussarps
left a comment
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.
Quasi bon pour moi.
- j'aurais plus mis la cli comme l'option 1 et le lancement via tsx comme l'option 2
- ça me paraîtrait bien de rajouter mention de la cli :
- dans le quickstart de la documentation "docusaurus"
- dans les guides au sein de la documentation "docusaurus"
- et la PR peut être l'occasion d'ajouter la méthode de lancement via le plugin npm que t'utilises en local pour simuler une bdd mongo, peut-être plus dans les options d'installation pour debuging local
|
|
||
| ${program.description()}. | ||
|
|
||
| ## Commands |
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.
j'aurais plus fait une page par commande comme le fait AWS, non ?
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.
Ça ferait des pages très vides
docusaurus/package.json
Outdated
| "start": "docusaurus start", | ||
| "build": "docusaurus build", | ||
| "start": "npm run build:cli && docusaurus start", | ||
| "build": "npm run build:cli && docusaurus build", |
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.
on peut garder ça dans build mais pas forcément dans "start" pour pouvoir continuer à lancer la doc facilement.
Sinon la partie tsdoc est décrite et générée via un plugin docusaurus - mais sans doute trop compliqué pour la cli en effet
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.
lancer la doc facilement
Est-ce vraiment moins facile ? C'est toujours la même commande, et une quinzaine de secondes en plus à tout casser. Au contraire, si tu ne mets pas le build:cli dans le start et que tu n'as jamais buildé la doc de la cli bah ça plante.
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.
Oui c'est moins facile, surtout dans le cas où je stoppe mon docusaurus et le relance en suivant.
Globalement c'est aussi ce que font les autres plugins, dont le plugin "typedoc" : il se compile au moment du build. Et par contre si le dossier "api" n'est pas là, le docusaurus ne plante pas.
En fait la méthode idoine ça serait de mettre la cli comme un plugin dans typedoc - on a déjà un plugin custom webflowIntegrationPlugin. Mais si on peut pas le faire, l'autre option, ça serait de garder le build de la cli le plus discret et économique possible
b3d61cd to
fd99685
Compare
Change description
Add cli documentation to the online documentation.
Type of change
Documentation
Related issues
Checklists
Development
Security
Code review