Skip to content

Add more expansive #db.f cursor documentation#573

Merged
matr1x-hackmud merged 11 commits into
comcode-org:mainfrom
Fayti1703:db/refine-cursor
Apr 2, 2026
Merged

Add more expansive #db.f cursor documentation#573
matr1x-hackmud merged 11 commits into
comcode-org:mainfrom
Fayti1703:db/refine-cursor

Conversation

@Fayti1703
Copy link
Copy Markdown
Contributor

Problem

The documentation for #db.f's cursors is:

  • hard to find
  • missing specific examples
  • missing several important details (e.g. limit+count interactions)

Context

This is a first draft.

First draft of db cursor refinement.

Expect this commit to VANISH in the future.
Copy link
Copy Markdown
Member

@matr1x-hackmud matr1x-hackmud left a comment

Choose a reason for hiding this comment

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

Looks great for a first draft! Minus two things I've caught in this current PR I think it could just go in

Comment thread docs/scripting/db/cursor/index.mdx Outdated
Comment thread docs/scripting/db/cursor/count.mdx
@CLAassistant
Copy link
Copy Markdown

CLAassistant commented Apr 2, 2026

CLA assistant check
All committers have signed the CLA.

@Fayti1703 Fayti1703 marked this pull request as ready for review April 2, 2026 21:19
@Fayti1703
Copy link
Copy Markdown
Contributor Author

Note: I have signed the CLA, but either the CLA Assistant or GitHub seem to have had a network blip shortly after, so the check is still marked as pending. I cannot run a recheck myself (the link just redirects to the GitHub home page).

@matr1x-hackmud
Copy link
Copy Markdown
Member

matr1x-hackmud commented Apr 2, 2026

Looks like the CLA went through.

I have re-reviewed all the changes in a code space and it looks fine to me. Thank you very much for your contribution!

@matr1x-hackmud matr1x-hackmud merged commit b8a6a0d into comcode-org:main Apr 2, 2026
4 checks passed
@Fayti1703 Fayti1703 deleted the db/refine-cursor branch April 2, 2026 21:29
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