Plots2: FEATURE: Integrate automatic code style checker

[VladimirMikulic]

Since I've been an active member of this community I see a lot (to be honest all) of beginners make the same styling mistakes and every time I have to point it out to them to enforce clean codebase.
I believe that other mentors could agree with me as well.

It would be very beneficial to the project and us (mentors) to have an automatic code style checker like Stickler CI. The checker would save the enormous amount of our time and make sure that the code is properly formatted.

@jywarren @SidharthBansal @nstjean @sashadev-sky @cesswairimu your views on this? Thanks.

[cesswairimu]

@VladimirMikulic Wow :rocket: :rocket: that is a great idea... love it :tada: :tada: I don't know anything about Sticker CI though... I will check it out

[VladimirMikulic]

Thanks, @cesswairimu. It's nothing complicated. We just specify formatting configurations for each language that we want to check and that's it. I am not the owner of the repo, so I can not set it up myself, but I'll be more than happy to assist admins :)

[sashadev-sky]

sounds fine by me! looping in @gauravano

[Tlazypanda]

@VladimirMikulic That's an awesome idea :rocket:

[nstjean]

I love the idea of having consistent code!!

[grvsachdeva]

Hi everyone!

@VladimirMikulic great idea. I think a few things we need to check about the Strickler are - how easy it is to use, how it present errors (this one is a bit important, as contributors will get to learn a lot if it highlights the rule).

Also, Strickler CI internally used rubocop for checking and fixing issues. I think we should check if codeclimate which we currently use can provide the same service (not in terms of automatic fixing but highlighting the issues in review).

For testing and comparing @VladimirMikulic you can install the app on your forked repo and after that, it can be installed here.

[jywarren]

I'm just noting that i reconfigured CodeClimate to leave in-line suggestions and it is pretty nice! It is limited to 5, i think, so it's not overwhelming. It looks like this:

Maybe this is a nice way for this to work as well?

https://github.com/publiclab/plots2/pull/8527#pullrequestreview-506101602