jhipster / jhipster-lite

JHipster Lite ⚡ is a development platform to generate, develop & deploy modern web applications & microservices architecture, step by step - using Hexagonal Architecture :gem:
https://lite.jhipster.tech
Apache License 2.0
450 stars 206 forks source link

Update readme when a module is execute #1933

Closed antarus closed 2 years ago

antarus commented 2 years ago

When adding an option it might be good to update the readme on how to configure/use it. For example when adding a database, it might be good to add the command to use to launch the docker-compose.

pascalgrimaud commented 2 years ago

Agree, the ReadmeService has been created, and needed to be called to add text in README for each option

DamnClin commented 2 years ago

What I did for the cucumber was was to create a dedicated documentation in /documentation/cucumber.md. The goal is that it will be easier to update.

I'm not a fan of updating the readme if we want to keep the possibility to start JHLite on a project not generated with JHLite...

We can also build something that will:

This way it'll work in all cases. I can add it to the module mechanism if you want

pascalgrimaud commented 2 years ago

Another / similar idea would be:

antarus commented 2 years ago

I did an implementation for postgresql but I think a mix of your ideas would be ideal. In addition, it would limit my problem of separation between documentation for development and configuration in the readme. All the documentation for the module would be in one place.

Adding this possibility to the module seems to be the right thing to do

DamnClin commented 2 years ago

I created https://github.com/jhipster/jhipster-lite/issues/1952 that will help to achieve this. Dunno when but I'm gonna do an implementation for this optional replacement thing :)

DamnClin commented 2 years ago

I ended up doing this https://github.com/jhipster/jhipster-lite/pull/1955 once this is merged the module documentation feature should be easier to do. @antarus you are working on it or I add it to the modules mechanism?

antarus commented 2 years ago

You can add it. i can use your work after ;)

DamnClin commented 2 years ago

I ended up adding it here https://github.com/jhipster/jhipster-lite/pull/1955

antarus commented 2 years ago

@DamnClin I started looking to refactor the postgresql generator with the new module system. I think it lacks the management of docker images or does it use the current DockerApplicationService service? Sorry, I'm starting with the hexagonal architecture and DDD in general and I wonder if the docker generator should not be included in the module system :D

DamnClin commented 2 years ago

Hello,

Modules still don't have docker management :). I can take a look at it, no idea of what's needed for that...

DamnClin commented 2 years ago

I may be wrong (just took 3mn to check that) but I don't think there is a need for a specific docker management in module since module is not doing anything special with docker file or the likes. I think you can rely on the existing service to get the version and do the update you want on files. Am I missing something there?

antarus commented 2 years ago

Ok I will look at how to use the existing service with the new module system. Docker files can be managed via the module's template system too.
Thank you

DamnClin commented 2 years ago

Tell me if I missed something, that's totally possible :)