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.

Sign up for GitHub

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

Jump to bottom

global::operation() or delta::operation() #140

Closed
saiichihashimoto opened this issue Aug 26, 2023 · 1 comment
Closed

global::operation() or delta::operation() #140

saiichihashimoto opened this issue Aug 26, 2023 · 1 comment

Comments

@saiichihashimoto
Copy link
Contributor

sanity shows the operation() function under the delta namespace

image

The GROQ spec shows the operation function under the global namespace

image

groq-js has it under the delta namespace

image

What's the truth? And can we get this all consolidated?
@saiichihashimoto saiichihashimoto mentioned this issue Aug 26, 2023
Closed
@judofyr
Copy link
Collaborator

judofyr commented Sep 5, 2023

groq-js is correct, the spec has a typo here.

@judofyr judofyr closed this as completed Sep 5, 2023
@saiichihashimoto saiichihashimoto mentioned this issue Sep 5, 2023
Merged
saiichihashimoto added a commit to saiichihashimoto/sanity-typed that referenced this issue Sep 5, 2023
@saiichihashimoto
fix(functions): operation is in the delta namespace
5766f96 
sanity-io/groq-js#140
sanity-io/GROQ#111
sanity-io/GROQ#114
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@judofyr @saiichihashimoto