-
Notifications
You must be signed in to change notification settings - Fork 302
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
Languages: added Rust [WIP] #3419
base: master
Are you sure you want to change the base?
Conversation
Signed-off-by: Defman21 <[email protected]>
Linting:
Integrations:
|
Signed-off-by: Defman21 <[email protected]>
Signed-off-by: Defman21 <[email protected]>
@Naatan since cargo lints the whole project (and you can't lint a file outside of the project because of |
@Defman21 I would go so far as to say it linting the entire project is a deal breaker. It's unlikely we'd merge it into Komodo if that's the case. Linting happens continuously as you type, so having it spin off a super expensive lint each time would be problematic to say the least. Regardless, thanks for the contribution! :) So you know UDL now? ;) |
Okay, I'll delete the entire linting then. And yes, I know UDL now :) |
Actually, after thinking a bit more, I think I'll tweak the linting so it will ignore some of the errors related to import, binary/library based things such as missing |
Hmm, Rust language server supports linting, so I might use it instead of compiling files. |
CodeIntel is something I really want to implement, but the API is closed atm.