Skip to content

fix: grammar and articles in function and path expression docs#9090

Merged
orizi merged 3 commits intostarkware-libs:mainfrom
anim001k:main
Dec 15, 2025
Merged

fix: grammar and articles in function and path expression docs#9090
orizi merged 3 commits intostarkware-libs:mainfrom
anim001k:main

Conversation

@anim001k
Copy link
Contributor

  • Fixes article usage: An xref:constant-items... in path-expressions.adoc.
  • Improves wording: See an example of calling it below in functions.adoc.

@reviewable-StarkWare
Copy link

This change is Reviewable

Copy link
Collaborator

@orizi orizi left a comment

Choose a reason for hiding this comment

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

@orizi reviewed 1 of 2 files at r1, all commit messages.
Reviewable status: 1 of 2 files reviewed, 1 unresolved discussion (waiting on @anim001k)


docs/reference/src/components/cairo/modules/language_constructs/pages/path-expressions.adoc line 7 at r1 (raw file):

* A local variable. In this case the path is a single identifier.
* An xref:constant-items.adoc[constant item].

this is wrong.

@anim001k anim001k requested a review from orizi December 14, 2025 19:45
Copy link
Collaborator

@orizi orizi left a comment

Choose a reason for hiding this comment

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

:lgtm:

@orizi reviewed 1 of 1 files at r2, all commit messages.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on @anim001k)

@orizi orizi enabled auto-merge December 15, 2025 02:48
@orizi orizi added this pull request to the merge queue Dec 15, 2025
Merged via the queue into starkware-libs:main with commit 68c5cff Dec 15, 2025
54 checks passed
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