hintman icon indicating copy to clipboard operation
hintman copied to clipboard

[RFC] Rust support

Open popzxc opened this issue 5 years ago • 3 comments
trafficstars

I'm not sure whether it's applicable for this project, but I would love it if this bot will be not haskell-targeted and there will be support of other programming languages linting.

In particular, I'm interested in rust:

  • rustfmt checks for appropriate formatting;
  • rust-clippy checks for code quality.

Adding of this feature support doesn't seem to be hard in general (those tools can output JSON), but this will cause the following drawbacks:

  • rust toolchain (which includes rustup, rustc, rust-std, cargo, rustfmt and clippy) will be required to be installed for testing. This can be done in two commands (rustup bootstraps everything on its own), but I'm not sure whether it's rationale to make project maintainers depend on the additional language.
  • there seems to be no deployment script in the repo, so I assume that deployment is made by hands, and this feature will require the responsible person to install rust on the deployment server, which I can't help with (except for consulting, of course).

So @chshersh / @vrom911 can you please tell whether you like this feature or not? If you do, I'll be happy to implement it.

This feature depends on #110.

popzxc avatar Dec 14 '19 04:12 popzxc

Wow, this is a cool idea :rocket:

Initially, we wanted to make it be general-purpose, so such proposals completely suit the purposes of the hintman.

I think we would need to think about the config first as you mentioned and to decide with the format so that it would be possible to setup up it for other languages and extend if necessary.

Sorry, if it would take some time to decide on that, as we are a bit short on time at the moment, but it should go faster after the holidays :slightly_smiling_face: Also, as none of us knows Rust, it could probably take more time to get into Rust tools and how to use it. Currently, we are using HLint as a library, which makes things easier a bit.

vrom911 avatar Dec 18 '19 10:12 vrom911

@vrom911 Thanks for writing this! These are my thoughts as well. I only want to add that we deploy Hintman to heroku using thestack buildpack:

  • https://github.com/mfine/heroku-buildpack-stack

chshersh avatar Dec 18 '19 10:12 chshersh

Also, as none of us knows Rust, it could probably take more time to get into Rust tools and how to use it.

I'm always open for guiding/assistance, if needed. Also, once configuration issue will be solved, I can provide (at least initial/draft) implementation, of course.

Sorry, if it would take some time to decide on that, as we are a bit short on time at the moment, but it should go faster after the holidays

Sure! We aren't bothered with deadlines, are we? :wink:

popzxc avatar Dec 18 '19 10:12 popzxc