Skip to content
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: Add missing KDocs for exposed-core vendors API #1892

Merged
merged 6 commits into from
Nov 24, 2023

Conversation

bog-walk
Copy link
Member

Covers files in the ops and vendors packages in the exposed-core module.

All public API elements now show a KDoc when hovered over by a mouse in IDE or when Quick Documentation shortcut is used (F1 | Ctrl+Q).

@bog-walk bog-walk requested review from e5l and joc-a November 23, 2023 02:44
Add KDocs to InListOps and Vendor package files.
Add KDocs to base statement classes.
Add KDocs to final files in vendors package.
Swap any use of 'command' with either 'statement' or 'query'.
@bog-walk bog-walk force-pushed the bog-walk/add-missing-core-kdocs branch from 5e8f7c3 to 98f80ac Compare November 24, 2023 18:53
@bog-walk bog-walk merged commit 84723ce into main Nov 24, 2023
5 checks passed
@bog-walk bog-walk deleted the bog-walk/add-missing-core-kdocs branch November 24, 2023 20:49
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