Skip to content

Conversation

@jrentlez
Copy link
Contributor

See README.md

@jrentlez jrentlez self-assigned this Nov 10, 2025
@jrentlez jrentlez marked this pull request as draft November 10, 2025 12:20
Copy link
Member

@georg-schwarz georg-schwarz left a comment

Choose a reason for hiding this comment

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

Good job!

I left some quite nitpicky comments about wording, not the design (which is on point IMO).

Co-authored-by: Georg Schwarz <dev@georg-schwarz.com>
@jrentlez jrentlez marked this pull request as ready for review November 13, 2025 14:04
Copy link
Member

@georg-schwarz georg-schwarz left a comment

Choose a reason for hiding this comment

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

LGTM. Please don't forget to change the status of the RFC before merging :-)

Btw, I noticed we introduce an inconsistency with this RFC:

  • access to members via row[<number/text>] brackets
  • access to text length via value.length in constraints

@jrentlez jrentlez merged commit 0cf634c into main Nov 18, 2025
4 checks passed
@jrentlez jrentlez deleted the rfc20-table-schema-value-type branch November 18, 2025 13:54
@github-actions github-actions bot locked and limited conversation to collaborators Nov 18, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants