Remove NeogitCommitMessage filetype

[vinibispo]

Close #405

• Remove NeogitCommitMessage filetype
• Turn it back to gitcommit filetype

[gegoune]

I am not entirely sure but those probably can just be removed. Nvim comes with gitcommit syntax as well as there is treesitter parser for gitconmit now.

[vinibispo]

I'll try removing

[CKolkey]

Yea, these can be removed altogether. Incorrect. See below.

[CKolkey]

An update on this: These should not be removed, and should be left unchanged. I was mistaken before: they have nothing to do with the NeogitGitCommit popup. These three lines are just for syntax highlighting on the status buffer of the branch name/remote name.

[CKolkey]

This change would prevent lua/neogit/autocmds.lua:19 from working properly. Not sure the consequences of that.

[gegoune]

      if api.nvim_buf_get_option(o.buf, "filetype"):find("Neogit") then
        return
      end

This could/should also check for gitcommit in that case. Not sure what implications of not checking that are, refreshing Neogit ongitcommit buffers' save makes little sense here I think.

[CKolkey]

Yea. I've been running with this patch for a week or two now without issue, and without addressing the above autocmd.

[vinibispo]

I'll fix it, thanks @gegoune and @CKolkey

[vinibispo]

Sorry guys, I'll promise that I'll fix it