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

Improve jhipster cli global/local messages. #17836

Merged
merged 1 commit into from
Feb 14, 2022

Conversation

mshima
Copy link
Member

@mshima mshima commented Feb 9, 2022

  • cli:
    • change message from Using JHipster version installed globally to Using bundled JHipster
    • change message from Using JHipster version installed locally in current project's node_modules to Switching to JHipster installed locally in current project's node repository (node_modules)
    • print Using bundled JHipster if the current jhipster is already in current project's repository. Ex: npx jhipster
    • allow commands to set a custom useOptions to force some options.

Or current cli switching to current project repository jhipster is not node/npm default, so we need to improve our messages.


Please make sure the below checklist is followed for Pull Requests.

When you are still working on the PR, consider converting it to Draft (below reviewers) and adding skip-ci label, you can still see CI build result at your branch.

@Tcharl
Copy link
Contributor

Tcharl commented Feb 10, 2022

LGTM

@DanielFran DanielFran merged commit 8a31ec7 into jhipster:main Feb 14, 2022
@mshima mshima deleted the skip_ci-cli_messages branch February 17, 2022 09:33
@pascalgrimaud pascalgrimaud added this to the 7.7.0 milestone Feb 26, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants