-
-
Notifications
You must be signed in to change notification settings - Fork 199
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
docs: updated wrong section name in NAMING_FORMATTER in constraint docs for all languages #1688
Changes from 1 commit
b5b70cb
0298a30
2a7c98b
10a0fef
ce5d9bd
d22bc5c
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 |
---|---|---|
|
@@ -11,7 +11,7 @@ These are the constraints that is applied to model naming. The `Rule key` is wha | |
|NO_NUMBER_START_CHAR|No numbers as starting characters|Default behavior is pre pending `number_` in front of the first character| | ||
|NO_EMPTY_VALUE|No empty values|Default behavior is to use `empty` as name. | | ||
|NO_RESERVED_KEYWORDS|No reserved keywords|TypeScript has a list of reserved keywords ([see the full list here](../../src/generators/typescript/Constants.ts))| | ||
|NAMING_FORMATTER|Must be formatted equally|Model name is formatted using pascal case| | ||
|NAMING_FORMATTER|Must be formatted equally|Property name is formatted using pascal case| | ||
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. This section here is actually for the Model naming formatted 🙂 So this needs to be reverted. The other section is correct 👍 Remember to propagate it across languages 🙂 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. Hey @jonaslagoni, Thanks for the review! I made the necessary changes. |
||
|
||
## Property naming | ||
These are the constraints that is applied to object properties and the naming of them. The `Rule key` is what you can use in the options to overwrite the default behavior. See [constraint customization](./README.md#Customization). | ||
|
@@ -21,7 +21,7 @@ These are the constraints that is applied to object properties and the naming of | |
|NO_NUMBER_START_CHAR|No numbers as starting characters|Default behavior is pre pending `number_` in front of the first character| | ||
|NO_EMPTY_VALUE|No empty values|Default behavior is to use `empty` as name. | | ||
|NO_RESERVED_KEYWORDS|No reserved keywords|TypeScript has a list of reserved keywords ([see the full list here](../../src/generators/typescript/Constants.ts))| | ||
|NAMING_FORMATTER|Must be formatted equally|Model name is formatted using pascal case| | ||
|NAMING_FORMATTER|Must be formatted equally|Property name is formatted using pascal case| | ||
|NO_DUPLICATE_PROPERTIES|No duplicate properties|If any of the above constraints changes the property name, we must make sure that no duplicates exist within the same object. If any is encountered `reserved_` is pre-pended. This is done recursively until no duplicates are found.| | ||
|
||
|
||
|
@@ -34,7 +34,7 @@ These are the constraints that is applied to enum keys. The `Rule key` is what y | |
|NO_NUMBER_START_CHAR|No numbers as starting characters|Default behavior is pre pending `number_` in front of the first character| | ||
|NO_EMPTY_VALUE|No empty values|Default behavior is to use `empty` as name. | | ||
|NO_RESERVED_KEYWORDS|No reserved keywords|TypeScript has a list of reserved keywords ([see the full list here](../../src/generators/typescript/Constants.ts))| | ||
|NAMING_FORMATTER|Must be formatted equally|Model name is formatted using pascal case| | ||
|NAMING_FORMATTER|Must be formatted equally|Property name is formatted using pascal case| | ||
|NO_DUPLICATE_KEYS|No duplicate enum keys|If any of the above constraints changes the enum key, we must make sure that no duplicates exist within the same enum. If any is encountered `reserved_` is pre-pended. This is done recursively until no duplicates are found.| | ||
|
||
## Constant | ||
|
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.
This one here is for the enum keys and how they are formatted, not the property name 🙂 Same applies for all the constraints.