Skip to content

Comments

RFC 0021: Sheet row syntax#689

Merged
jrentlez merged 5 commits intomainfrom
rfc-0021
Feb 19, 2026
Merged

RFC 0021: Sheet row syntax#689
jrentlez merged 5 commits intomainfrom
rfc-0021

Conversation

@jrentlez
Copy link
Contributor

This RFC removes some unnecessary definitions of RFC 0020 regarding sheet row access.

Feedback is welcome, especially on the alternatives section.

@jrentlez
Copy link
Contributor Author

@georg-schwarz As discussed, I've changed the RFC to keep the SheetRow value-type.

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.

Thanks @jrentlez

Did you have time to look into Langium's class member scoping guide? Could this be an alternative to introducing an operator?

@jrentlez
Copy link
Contributor Author

Did you have time to look into Langium's class member scoping guide? Could this be an alternative to introducing an operator?

I think we would run into the familiar problem that we cannot access sheet columns with wierd names.

IMHO, if we plan to introduce class member scoping, we should first experiment with NestedPropertyAccess from expressions.langium instead of replacing this operator.

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.

I'm happy to proceed. Don't forget to update the metadata table in the RFC :)

@jrentlez jrentlez merged commit 0875fa8 into main Feb 19, 2026
4 checks passed
@jrentlez jrentlez deleted the rfc-0021 branch February 19, 2026 10:28
@github-actions github-actions bot locked and limited conversation to collaborators Feb 19, 2026
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.

2 participants