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

Print a warning about .clj files that are not linted, and why they are not #102

Closed
jafingerhut opened this issue Nov 7, 2014 · 3 comments

Comments

@jafingerhut
Copy link
Collaborator

My dolly project's command 'lein dolly ns' warns about files that are not analyzed because no ns form was found. Eastwood should do something similar, and perhaps also mention that .cljs and .cljx files are currently ignored, if any are found.

@jafingerhut
Copy link
Collaborator Author

Related to how issue #101 is handled -- if an option is created to try to lint .clj files that are by default ignored, obviously we shouldn't warn about them then, but maybe will want to print out some message about how they are being handled differently.

@jafingerhut jafingerhut changed the title Print a warning about .clj files that are not linter, and why they are not Print a warning about .clj files that are not linted, and why they are not Nov 7, 2014
@jafingerhut
Copy link
Collaborator Author

These should be reported as a separate lint warning map for each such file, similar to other linters, for easier handling by potential code that post-processes warnings somehow. They should have their own linter name, which can be disabled if one doesn't want to see these warnings.

Similarly, the current filename / namespace warnings should be reported as warning maps, too, with their own linter name, but in that case they cannot be disabled.

@jafingerhut
Copy link
Collaborator Author

Added with this commit: a3d7354

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