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

Editorial: Replace Get+Call by Invoke #1314

Closed
ptomato opened this issue Jan 20, 2021 · 2 comments · Fixed by #1361
Closed

Editorial: Replace Get+Call by Invoke #1314

ptomato opened this issue Jan 20, 2021 · 2 comments · Fixed by #1361
Assignees
@cjtenny
Labels
spec-text Specification text involved
Milestone
Next

Comments

@ptomato
Copy link
Collaborator

ptomato commented Jan 20, 2021

Raised by @ljharb in #1311. Use Invoke in the spec text instead of Get immediately followed by Call (although take care not to change a single Get followed by multiple Call because that will change the observable effects)
@ptomato ptomato added the spec-text Specification text involved label Jan 20, 2021
@ptomato ptomato added this to the Stage 3 milestone Jan 20, 2021
@littledan
Copy link
Member

These editorial issues can be resolved by Stage 4 in my opinion.

@ptomato ptomato modified the milestones: Stage 3, Next Jan 22, 2021
@ljharb
Copy link
Member

ljharb commented Jan 22, 2021

Certainly can be, but needn't wait either.

cjtenny added a commit that referenced this issue Feb 11, 2021
@cjtenny
Replace Get+Call by Invoke
75fee9b 
Fixes #1314.
@cjtenny cjtenny mentioned this issue Feb 11, 2021
Merged
cjtenny added a commit that referenced this issue Feb 11, 2021
@cjtenny
Replace Get+Call by Invoke (#1361)
ef8e60d 
Fixes #1314.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@cjtenny cjtenny

Labels
spec-text Specification text involved
Projects
None yet
Milestone
Next
Development

Successfully merging a pull request may close this issue.

Replace Get+Call by Invoke tc39/proposal-temporal
4 participants
@ljharb @littledan @ptomato @cjtenny