tpope / vim-fugitive

fugitive.vim: A Git wrapper so awesome, it should be illegal
https://www.vim.org/scripts/script.php?script_id=2975
20.03k stars 1.01k forks source link

Error detected while processing modelines (regression since "Use callback based workflow for status buffer") #2334

Closed liskin closed 2 months ago

liskin commented 2 months ago

It seems vim's trying to execute modelines in commit messages since 8c8cdf4405cb8bdb70dd9812a33bb52363a87dbc:

$ mkdir git1
$ git -C git1 init
$ git -C git1 commit --allow-empty 'initial'
$ git clone ./git1 ./git2 # make a clone so we get to see the "Unpushed to origin/main" section
$ git -C git2 commit --allow-empty -m 'vim: Update'
$ cd git2
$ vim -c Git
…
Error detected while processing modelines:
line    6:
E518: Unknown option: Update
tpope commented 2 months ago

Fixed in https://github.com/tpope/vim-fugitive/pull/2335.

liskin commented 2 months ago

Oh, indeed, and apologies for missing the existing report. Must've been tired, or… something :-/