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.

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

add support for proposal function partial application #1205

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

morr
Copy link

@morr morr commented May 3, 2020

@morr morr changed the title add support for proposal-partial-application operator add support for proposal-partial-application May 3, 2020
@morr morr changed the title add support for proposal-partial-application add support for proposal function partial application May 3, 2020
@vieira
Copy link
Contributor

vieira commented May 21, 2021

It seems it would not work for this case?

1 + 1
  |> f(1, ?)

vieira added a commit to vieira/vim-javascript that referenced this pull request May 22, 2021
As defined in tc39/proposal-partial-application, proposed in
pangloss#1205
@vieira
Copy link
Contributor

vieira commented May 22, 2021

This seems to work for both cases and does not highlight the following , or ) as part of the ?.

syntax match jsFuncArgPartApp contained /\%([(,]\s*\)\@<=?\%(\s*[,)]\)\@=/ containedin=jsFuncArgs

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.

2 participants