Skip to content

Align with entity naming best practice - #2201

Merged
renejeglinsky merged 1 commit into
mainfrom
pluralise-entity-names-in-cdl
Dec 19, 2025
Merged

Align with entity naming best practice#2201
renejeglinsky merged 1 commit into
mainfrom
pluralise-entity-names-in-cdl

Conversation

@qmacro

@qmacro qmacro commented Nov 12, 2025

Copy link
Copy Markdown
Collaborator

See:
https://cap.cloud.sap/docs/guides/domain-modeling#naming-conventions
There are plenty of entity names in examples in this CDL topic. Some of
them are "real" in that they are genuine representations such as Orders
and Products. These should be subject to the naming convention best
practice of being pluralised.

Others are clearly illustrative only, such as the Foo, Bar, Baz set and
those conveying meaning for the given example such as EmployeeView,
SomeView, P, Proj, SomeExposedEntity and so on. These should not be
subject to the naming convention.

Note: Perhaps we should add something to the naming convention section
itself to qualify the boundary of where it applies, i.e. "mostly" at the
schema level, less so at the service definition level?

See:
https://cap.cloud.sap/docs/guides/domain-modeling#naming-conventions
There are plenty of entity names in examples in this CDL topic. Some of
them are "real" in that they are genuine representations such as Orders
and Products. These should be subject to the naming convention best
practice of being pluralised.

Others are clearly illustrative only, such as the Foo, Bar, Baz set and
those conveying meaning for the given example such as EmployeeView,
SomeView, P, Proj, SomeExposedEntity and so on. These should not be
subject to the naming convention.

Note: Perhaps we should add something to the naming convention section
itself to qualify the boundary of where it applies, i.e. "mostly" at the
schema level, less so at the service definition level?
@renejeglinsky
renejeglinsky merged commit 1c283b8 into main Dec 19, 2025
9 checks passed
@renejeglinsky
renejeglinsky deleted the pluralise-entity-names-in-cdl branch December 19, 2025 11:03
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.

3 participants