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

runtime(vim): Improve :let-heredoc syntax highlighting #12923

Merged

Conversation

dkearns
Copy link
Contributor

@dkearns dkearns commented Aug 26, 2023

"trim" and "eval" are allowed in any order.

@cecamp, I sent this to you about a year ago but didn't receive a reply. This is unusual so I'm submitting the change here as well in case my email has dissipated in the ether.

@dkearns dkearns requested a review from cecamp as a code owner August 26, 2023 11:03
@codecov
Copy link

codecov bot commented Aug 26, 2023

Codecov Report

Merging #12923 (da55133) into master (4b0018f) will decrease coverage by 0.17%.
Report is 11 commits behind head on master.
The diff coverage is n/a.

❗ Current head da55133 differs from pull request most recent head dbef1e3. Consider uploading reports for the commit dbef1e3 to get more accurate results

@@            Coverage Diff             @@
##           master   #12923      +/-   ##
==========================================
- Coverage   82.20%   82.04%   -0.17%     
==========================================
  Files         160      160              
  Lines      196942   194679    -2263     
  Branches    44128    43696     -432     
==========================================
- Hits       161893   159719    -2174     
+ Misses      22138    22087      -51     
+ Partials    12911    12873      -38     
Flag Coverage Δ
huge-clang-Array ?
huge-clang-none 82.66% <ø> (?)
linux 82.66% <ø> (-0.16%) ⬇️
mingw-x64-HUGE 76.59% <ø> (+0.24%) ⬆️
mingw-x86-HUGE 77.08% <ø> (-0.28%) ⬇️
windows 78.18% <ø> (-0.23%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

see 92 files with indirect coverage changes

@dkearns dkearns force-pushed the runtime-vim-improve-letheredoc-highlighting branch from 2aba0ec to da55133 Compare August 26, 2023 11:24
@yegappan yegappan added this to the vim-9.1 milestone Aug 26, 2023
@chrisbra
Copy link
Member

@cecamp please

"trim" and "eval" are allowed in any order and whitespace is not
required after "=<<".
@dkearns dkearns force-pushed the runtime-vim-improve-letheredoc-highlighting branch from da55133 to dbef1e3 Compare November 11, 2023 16:03
@dkearns
Copy link
Contributor Author

dkearns commented Nov 11, 2023

I think this can be included @chrisbra, it's an isolated change.

@chrisbra
Copy link
Member

thanks, makse sense to include

@chrisbra chrisbra merged commit 9358b8d into vim:master Nov 11, 2023
31 of 32 checks passed
zeertzjq added a commit to zeertzjq/neovim that referenced this pull request Nov 11, 2023
runtime(vim): Improve :let-heredoc syntax highlighting (vim/vim#12923)

"trim" and "eval" are allowed in any order and whitespace is not
required after "=<<".

vim/vim@9358b8d

Co-authored-by: dkearns <dougkearns@gmail.com>
zeertzjq added a commit to neovim/neovim that referenced this pull request Nov 12, 2023
runtime(vim): Improve :let-heredoc syntax highlighting (vim/vim#12923)

"trim" and "eval" are allowed in any order and whitespace is not
required after "=<<".

vim/vim@9358b8d

Co-authored-by: dkearns <dougkearns@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants