Skip to content

Update cds-utils.md - #2896

Merged
chgeo merged 3 commits into
mainfrom
daogrady-patch-1
Sep 18, 2026
Merged

chgeo merged 3 commits into
mainfrom
daogrady-patch-1

Conversation

@daogrady

Copy link
Copy Markdown
Contributor

Note that beside the one-letter units, the implementation actually allows for a myriad of variations. But I think listing them all brings no additional value here. They are fully listed and will be code-completed and type checked if cds-types is installed. https://github.com/cap-js/cds-types/pull/601/changes#diff-541cab5af5e2e221456c34f99240d4ef78840c99fa13867b30d5b35b092a1f7cR5-R10

Note that beside the one-letter units, the implementation actually allows for a myriad of variations. But I think listing them all brings no additional value here. They are fully listed and will be code-completed and type checked if cds-types is installed.
https://github.com/cap-js/cds-types/pull/601/changes#diff-541cab5af5e2e221456c34f99240d4ef78840c99fa13867b30d5b35b092a1f7cR5-R10
@hyperspace-pr-bot

hyperspace-pr-bot Bot commented Sep 17, 2026

Copy link
Copy Markdown
Contributor

Control Panel

Hi, I'm an AI-powered Review Bot that helps you with summarizing and reviewing pull requests.
To interact with me, just use the following actions:

  • 📝 Summarize PR
  • 🔍 Review
  • 🗑️ Delete all bot comments and reviews

@hyperspace-pr-bot hyperspace-pr-bot Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I reviewed the changed documentation and left two inline suggestions focused on clarity and naming in the new ms4 section. Consistency checked: I did not find existing repository references for ms4 or the new wording pattern in the searched areas.

PR Bot Information

Version: 1.31.43

Comment thread node.js/cds-utils.md Outdated
Comment thread node.js/cds-utils.md Outdated
daogrady and others added 2 commits September 17, 2026 14:48
Co-authored-by: hyperspace-pr-bot[bot] <209611008+hyperspace-pr-bot[bot]@users.noreply.github.com>
Co-authored-by: hyperspace-pr-bot[bot] <209611008+hyperspace-pr-bot[bot]@users.noreply.github.com>
@chgeo
chgeo added this pull request to the merge queue Sep 18, 2026
Merged via the queue into main with commit 0461b8c Sep 18, 2026
4 checks passed
@chgeo
chgeo deleted the daogrady-patch-1 branch September 18, 2026 06:23
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.

2 participants