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

Update linters and fixers to apply command formatting more #3325

Closed
w0rp opened this issue Aug 28, 2020 · 1 comment
Closed

Update linters and fixers to apply command formatting more #3325

w0rp opened this issue Aug 28, 2020 · 1 comment

Comments

@w0rp
Copy link
Member

w0rp commented Aug 28, 2020

Now that #2556 has been implemented, it's important that linters and fixers generally use %s and %t formatting with filename-modifiers like %s:h for a file's directory name, and so on. The more linters use this kind of formatting, the more they will be capable of running in Docker, in virtual machines, or on other computers. We should go over all of the existing linters and fixers and update them to work this way.

@w0rp
Copy link
Member Author

w0rp commented Aug 28, 2020

I've applied the formatting wherever I safely could to existing linters now. There are quite a few linters that will require some careful adjustments if we want to support running them with some other file system, but we can handle those some other day.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant