[vim/vim] Update syntax/dosini.vim to the latest version (PR #12764)

24 views
Skip to first unread message

Hong Xu

unread,
Aug 11, 2023, 12:31:58 AM8/11/23
to vim/vim, Subscribed

The latest version is in https://github.com/xuhdev/syntax-dosini.vim/blob/master/syntax/dosini.vim. I've sent the file to Bram at the end of June, unfortunately it wasn't able to make into the Vim repository. RIP!


You can view, comment on, or merge this pull request online at:

  https://github.com/vim/vim/pull/12764

Commit Summary

  • ea6855f Update syntax/dosini.vim to the latest version

File Changes

(1 file)

Patch Links:


Reply to this email directly, view it on GitHub.
You are receiving this because you are subscribed to this thread.Message ID: <vim/vim/pull/12764@github.com>

codecov[bot]

unread,
Aug 11, 2023, 11:54:07 AM8/11/23
to vim/vim, Subscribed

Codecov Report

Merging #12764 (ea6855f) into master (81b8bf5) will decrease coverage by 5.49%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master   #12764      +/-   ##
==========================================
- Coverage   82.10%   76.61%   -5.49%     
==========================================
  Files         160      150      -10     
  Lines      193722   151408   -42314     
  Branches    43500    39030    -4470     
==========================================
- Hits       159046   115994   -43052     
- Misses      21825    23389    +1564     
+ Partials    12851    12025     -826     
Flag Coverage Δ
huge-clang-none ?
linux ?
mingw-x64-HUGE 76.61% <ø> (-0.01%) ⬇️
mingw-x86-HUGE ?
windows 76.61% <ø> (-1.60%) ⬇️

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

see 156 files with indirect coverage changes


Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.Message ID: <vim/vim/pull/12764/c1675007659@github.com>

Christian Brabandt

unread,
Aug 11, 2023, 1:29:53 PM8/11/23
to vim/vim, Subscribed

thanks.


Reply to this email directly, view it on GitHub.

You are receiving this because you are subscribed to this thread.Message ID: <vim/vim/pull/12764/c1675140725@github.com>

Christian Brabandt

unread,
Aug 11, 2023, 1:29:54 PM8/11/23
to vim/vim, Subscribed

Merged #12764 into master.


Reply to this email directly, view it on GitHub.

You are receiving this because you are subscribed to this thread.Message ID: <vim/vim/pull/12764/issue_event/10072283151@github.com>

Christian Brabandt

unread,
Aug 20, 2023, 4:58:46 AM8/20/23
to vim/vim, Subscribed

@xuhdev I think this patch introduced a slight regression. Can you please have a look at the patch posted here: https://groups.google.com/g/vim_use/c/9zccgo_RIqM/m/4OOjewNOBgAJ

Do you want to incorporate this into your version first or can I go ahead and merge it?

Thanks,
Chris


Reply to this email directly, view it on GitHub.

You are receiving this because you are subscribed to this thread.Message ID: <vim/vim/pull/12764/c1685229656@github.com>

Hong Xu

unread,
Aug 20, 2023, 8:19:01 PM8/20/23
to vim/vim, Subscribed

On 8/20/23 01:58, Christian Brabandt wrote:
> @xuhdev <https://github.com/xuhdev> I think this patch introduced a slight regression. Can you please have a look at the patch posted here: https://groups.google.com/g/vim_use/c/9zccgo_RIqM/m/4OOjewNOBgAJ <https://groups.google.com/g/vim_use/c/9zccgo_RIqM/m/4OOjewNOBgAJ>

>
> Do you want to incorporate this into your version first or can I go ahead and merge it?
>

I'm good with either way. I've merged the PR you created. Feel free to port it to this repo.

Thanks


Reply to this email directly, view it on GitHub.

You are receiving this because you are subscribed to this thread.Message ID: <vim/vim/pull/12764/c1685439282@github.com>

Christian Brabandt

unread,
Aug 21, 2023, 12:53:54 AM8/21/23
to vim/vim, Subscribed

okay I have merged it in 6909639


Reply to this email directly, view it on GitHub.

You are receiving this because you are subscribed to this thread.Message ID: <vim/vim/pull/12764/c1685639212@github.com>

Reply all
Reply to author
Forward
0 new messages