-
-
Notifications
You must be signed in to change notification settings - Fork 272
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
Introduce a separate attribute for declaring a linter to only operate on the exact files #954
Comments
Related #244 ? Isn't specifying a suffix yagni bc we can just use the filename suffix? |
T.i. remove tempfile_suffix and instead expose lint_mode as a linter setting. |
A buffer doesn't have to be associated with a file, e.g. it hasn't been saved yet. Also, if you lint embedded code by taking it out of the surrounding file and putting it in a temporary file, you can't use the original file's suffix. That said, especially in a unix-like context, programs that work on a file rarely care about that file's name. If a program receives a file to lint, it should usually just lint it unless the linter's author has been smoking something. I don't think you need a temp file suffix at all in 99% of cases. I do seem to remember that some php linter or other will not lint files that don't have
This is an all around good idea. |
Okay, just to summarize what I know:
But we cannot deduce the mode from 'cmd' unless we introduce an alias ${real_file} for ${file_name}. EDITED to make it proposal like, not just a thought |
Currently Maybe support |
Dude, I forgot that the Sublime variable is actually Also: |
Either is fine with me, so I guess pick the shorter one? "tmp" is a widely-understood abbreviation. Alternatively, |
I'll take |
discussion moved to #1073 |
Currently, you can declare a linter as a "file-only" linter setting
tempfile_suffix
to'-'
. Its effects are that no temporary file is generated while modifying the view and instead the linter is only run on the saved file with its original path.This API is quite unintuitive and should be replaced with a separate attribute just for that feature.
Related: #810.
The text was updated successfully, but these errors were encountered: