-
-
Notifications
You must be signed in to change notification settings - Fork 4k
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
Fix some typos in markdown files #25275
Changes from 1 commit
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,6 +1,6 @@ | ||
# JHipster Blueprints | ||
|
||
Blueprints allows to add new features or change current features. | ||
Blueprints allows adding new features or change current features. | ||
|
||
## Creating a Blueprint | ||
|
||
|
@@ -43,6 +43,6 @@ For this reason try to _avoid exact version at blueprint dependencies_ and disab | |
- priorities names are static constants at the generator class. | ||
`get initializing() {}` -> `get [Generator.INITIALIZING]() {}` | ||
Motivation can be found at https://github.com/jhipster/generator-jhipster/blob/main/rfcs/3-jhipster-rfc-unambiguous-priorities.md. | ||
- many property migration can be warned by enabling `jhipster7Migration` feature at the constructor. | ||
- much property migration can be warned by enabling `jhipster7Migration` feature at the constructor. | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. much looks weird to me, not a native speaker. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I think "many" sounds better, but I'm also trying to understand what the sentence should say. Are you trying to say many properties can be migrated automatically, or the deprecated ones can be printed to the console? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Removed properties will work but printed to console. |
||
- needles are not implemented in the generator base anymore. | ||
They are injected at source object provided at some priorities and used like https://github.com/jhipster/generator-jhipster/blob/6373c9c76e57d01c4c1451a276f0d78bfbdd2c42/generators/spring-cache/generator.mts#L97-L101 |
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.