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

Multiple-letter violation codes are not displayed by Vim Syntastic syntax checker #6

Closed
myii opened this issue Apr 1, 2018 · 5 comments

Comments

@myii
Copy link

myii commented Apr 1, 2018

This is not a bug with your plugin but I'm raising this here for your awareness, since you use multiple-letter violation code(s).

I've filed an issue with vim-syntastic: vim-syntastic/syntastic#2163.

The resolution of that bug may be of interest to you. If not, please disregard this report, with my apologies.

@peterjc
Copy link
Owner

peterjc commented Apr 1, 2018

Thanks for the heads up - I personally don't use vim so hadn't run into this. See also https://gitlab.com/pycqa/flake8/issues/339 for better documenting the prefix recommendations for flake8 plugin authors.

@myii
Copy link
Author

myii commented Apr 1, 2018

@peterjc You're welcome. Appreciate your plugin, by the way.

See also https://gitlab.com/pycqa/flake8/issues/339 for better documenting the prefix recommendations for flake8 plugin authors.

Brilliant, this is one of the resources that I couldn't track down when I was discussing violation code clashes recently @ PyCQA/flake8-bugbear#37.

Peter Cock @pjacock commented 9 months ago

I've opened #339 looking at the prefix conventions and improving the plugin development documentation about this.

Given since v3.0 Flake8 isn't restricted to 4 character long violation codes, I like your suggestion of using RST as the prefix here.

...

  • At some point, I came across another discussion, that was very much in favour of using longer prefixes, since there aren't enough to go around all of the plugins that are now available -- unfortunately, I don't have time to track that down right now

@myii
Copy link
Author

myii commented Apr 2, 2018

The Syntastic issue has been resolved so I'll close this issue, too.

@myii myii closed this as completed Apr 2, 2018
@peterjc
Copy link
Owner

peterjc commented Apr 2, 2018

Thanks for your efforts - future vim users' lives should be a little smoother now.

@myii
Copy link
Author

myii commented Apr 2, 2018

@peterjc I didn't do anything spectacular. It's all about the developers and maintainers, that's where the real credit is due.

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

2 participants