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

Add support for force-insights option #14485

Merged

Conversation

SudharakaP
Copy link
Member

@SudharakaP SudharakaP commented Mar 27, 2021

Somehow we have missed supporting the force-insights option. This adds the force-insights option back to the generator. Otherwise passing this option results in a error;

sudharaka@sudharaka-XPS-8700:~/IdeaProjects/GeneratedApplications/jhipsterTest$ jhipster --force-insights
INFO! Using JHipster version installed globally
error: unknown option '--force-insights'
sudharaka@sudharaka-XPS-8700:~/IdeaProjects/GeneratedApplications/jhipsterTest$

JHipster Online is also effected by this error which I've opened a PR as well; jhipster/jhipster-online#291


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

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

SudharakaP added a commit to SudharakaP/jhipster-online that referenced this pull request Mar 27, 2021
SudharakaP added a commit to jhipster/jhipster-online that referenced this pull request Mar 27, 2021
* feat: add force-insights back in

This option needs to be supported. Ideally we should fix this in the generator.

* feat: disable stats until jhipster/generator-jhipster#14485 is fixed
SudharakaP added a commit to SudharakaP/jhipster-online that referenced this pull request Mar 27, 2021
Copy link
Member

@mshima mshima left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The option is correct.
The text should match --no-insight, so change it to Disable the insights.
IMO it could lose the the.

Somehow we have missed supporting the force-insights option. This adds the force-insights option back to the generator.
@SudharakaP SudharakaP force-pushed the feat/add-support-for-force-insights-option branch from d19b117 to e54849d Compare March 28, 2021 02:22
@SudharakaP
Copy link
Member Author

The option is correct.
The text should match --no-insight, so change it to Disable the insights.
IMO it could lose the the.

@mshima : Thanks for the review. I've just pushed that (remove the). Hope it's okay now. 😄

@mshima mshima merged commit 0ea550b into jhipster:main Mar 28, 2021
@SudharakaP SudharakaP deleted the feat/add-support-for-force-insights-option branch March 28, 2021 04:02
@pascalgrimaud pascalgrimaud added this to the v7.0.1 milestone Apr 1, 2021
SudharakaP added a commit to jhipster/jhipster-online that referenced this pull request Apr 2, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants