-
Notifications
You must be signed in to change notification settings - Fork 72
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
Drop Ruby #286
Comments
When I run repolinter, I pay attention to the axiom gated results. We could probably do just as well as linguist (for my needs at least) by looking at file extensions. |
Agreed. https://www.npmjs.com/package/linguist-js is also an option. Sounds like it's built on the rules for github linguist, but not the ruby code. Licensee wise, losing the axiom would lose the apache-notice rule and the license-detectable-by-licensee rule; though the latter is off by default. |
Losing the license check would be a shame, have to see if we can find a replacement for this package. |
Issue Report
Licensee and Linguist are used as axioms to allow some linting rules to only apply when the output of those tools match the license or language. Unfortunately, both are written in Ruby which doubles the tech-stack complexity of the project. I'm not aware of anyone actively using these axioms and so my proposal is that we drop these.
The text was updated successfully, but these errors were encountered: