exercism / DEPRECATED.rikki

A worker written in golang for automatically nitpicking exercism submissions.
30 stars 10 forks source link

[ruby] False positive with indentation validation? #37

Open abeger opened 6 years ago

abeger commented 6 years ago

This code got flagged for inconsistent indentation and I cannot figure out why:

http://exercism.io/submissions/7fe9078c64fd4cb5a1b2072270ab4a74

Is there something else wrong with the code and the message is misleading?

Insti commented 6 years ago

I can't see any problems with the indentation.

My guess is that the begin on line 27 that's causing it.

But there are other reasons why you don't want to be doing that.

kytrinyx commented 6 years ago

Agreed—we shouldn't be flagging that begin as inconsistent indentation.

Any chance you'd have time to write a failing test for this for https://github.com/JacobNinja/exercism-analysis? Having a minimal code example that proves exactly what the codeminer is unhappy about would be super helpful towards getting a fix in.

abeger commented 6 years ago

Sure I'll take a swing at it

abeger commented 6 years ago

As something of an aside, has there been any talk of using a custom Rubocop configuration for the Ruby analyzer? exercism-analysis hasn't been updated in 3 years and is getting a little out of date.

kytrinyx commented 6 years ago

@abeger That's a great point. I've used Rubocop to write some linters and do some static analysis, and it's a good tool.

We won't be using rikki- directly in the v2, rather we'll be writing some new tools for mentors. I think it would be worth considering Rubocop for the new tools.

abeger commented 6 years ago

How far away is v2? Would it be worth my time to experiment with making Rikki RuboCop-based?

kytrinyx commented 6 years ago

@abeger we don't actually know—we hope not too far, but my guess would be more than weeks. My gut sense is "probably worth holding off", though I will start thinking about what v2's rikki would look like so we could potentially start working on that.

stale[bot] commented 6 years ago

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.