docs: adds C++ constraint docs #1696
Conversation
✅ Deploy Preview for modelina canceled.
|
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
There was a problem hiding this comment.
Welcome to AsyncAPI. Thanks a lot for creating your first pull request. Please check out our contributors guide useful for opening a pull request.
Keep in mind there are also other channels you can use to interact with AsyncAPI community. For more details check out this issue.
Pull Request Test Coverage Report for Build 7340979409
💛 - Coveralls |
docs/constraints/Cplusplus.md
Outdated
| |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|C++ has a list of reserved keywords ([see the full list here](../../src/generators/cplusplus/Constants.ts))| | ||
| |NAMING_FORMATTER|Must be formatted equally|Model name is formatted using pascal case| |
There was a problem hiding this comment.
The model namings use toSnakeCase function not pascal case, please double check that the implementation and docs (for all the constraints) are matching: https://github.com/asyncapi/modelina/blob/cbf55179f37adaa9b67564b2fc022615d7c66095/src/generators/cplusplus/constrainer/EnumConstrainer.ts#L42C35-L42C46
|
|
@jonaslagoni Made some changes. Notice that only |
|
hey, @jonaslagoni can I become part of a contributor for doc, maintenance? |
|
@ankur0904 docs for sure, however maintenance is for those who are code owners 😄 If you want to become a codeowner read more here: #1148 |
|
@all-contributors please add @ankur0904 for docs |
|
I've put up a pull request to add @ankur0904! 🎉 |
|
Thanks @jonaslagoni |
|
🎉 This PR is included in version 3.0.0-next.11 🎉 The release is available on: Your semantic-release bot 📦🚀 |
|
🎉 This PR is included in version 2.1.2 🎉 The release is available on: Your semantic-release bot 📦🚀 |
Description
Adds the missing
C++constraint docs.Related Issue
Fix #1671