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

An idea of new rule: Must not use augroup in ftdetect/*.vim #264

Open
thinca opened this issue Jan 27, 2018 · 0 comments
Open

An idea of new rule: Must not use augroup in ftdetect/*.vim #264

thinca opened this issue Jan 27, 2018 · 0 comments

Comments

@thinca
Copy link

thinca commented Jan 27, 2018

:help ftdetect

Note that there is no "augroup" command, this has already been done
when sourcing your file.

refs:
This PR added augroup to ftdetect/gofiletype.vim according to vint's warning.
https://github.com/fatih/vim-go/pull/1558/files#diff-349dca49094d5651df28e5df61a4e0c9
This PR fixed it.
fatih/vim-go#1645

jparise added a commit to jparise/vim-graphql that referenced this issue Nov 25, 2018
This incorrectly complains about missing an `augroup` in a `ftdetect`
script. From `:help ftdetect`:

  Note that there is no "augroup" command, this has already been done
  when sourcing your file.  You could also use the pattern "*" and then
  check the contents of the file to recognize it.

... so I'm suppressing that warning in ftdetect/graphql.vim.

See also: Vimjas/vint#264
amadeus pushed a commit to pangloss/vim-javascript that referenced this issue Aug 14, 2019
This is from the help doc :help ftdetect:

Note that there is no "augroup" command, this has already been done
when sourcing your file.  You could also use the pattern "*" and then
check the contents of the file to recognize it.

Reference:
Vimjas/vint#264
fatih/vim-go#1645
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant