Skip to content

Conversation

@e-kwsm
Copy link
Contributor

@e-kwsm e-kwsm commented Nov 29, 2023

No description provided.

Copy link
Contributor

@dkearns dkearns left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the update.

It looks like this is maintained and distributed as part of the CMake project. If you're managing updates to the Vim distribution, rather than Dimitri, it might be a good idea to also list yourself as a maintainer in the header and add yourself to https://github.com/vim/vim/blob/master/.github/CODEOWNERS

@chrisbra might have some thoughts on how to manage this scenario.

@chrisbra
Copy link
Member

Thanks, I don't have anything more to add, you covered it all :)

@e-kwsm e-kwsm marked this pull request as draft December 1, 2023 05:19
@e-kwsm
Copy link
Contributor Author

e-kwsm commented Dec 4, 2023

The upstream is https://github.com/pboettch/vim-cmake-syntax.
Consequently this goes to pboettch/vim-cmake-syntax#33.

@chrisbra
Copy link
Member

chrisbra commented Dec 5, 2023

so is this ready? Or is there anything missing?

@e-kwsm e-kwsm force-pushed the cmake branch 2 times, most recently from 52849f3 to badfef0 Compare December 6, 2023 15:32
@e-kwsm e-kwsm marked this pull request as ready for review December 6, 2023 15:33
@e-kwsm
Copy link
Contributor Author

e-kwsm commented Dec 6, 2023

Ready but pboettch/vim-cmake-syntax#33 is not merged.

@chrisbra chrisbra merged commit 3afc9f2 into vim:master Dec 14, 2023
clason added a commit to clason/neovim that referenced this pull request Dec 15, 2023
runtime(cmake): sync runtime files with upstream (vim/vim#13597)

vim/vim@3afc9f2

Co-authored-by: Eisuke Kawashima <e-kwsm@users.noreply.github.com>
@e-kwsm e-kwsm deleted the cmake branch December 15, 2023 08:46
clason added a commit to neovim/neovim that referenced this pull request Dec 15, 2023
runtime(cmake): sync runtime files with upstream (vim/vim#13597)

vim/vim@3afc9f2

Co-authored-by: Eisuke Kawashima <e-kwsm@users.noreply.github.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.

4 participants