[Tests] Setup StandardRB for Ruby

[kalimar]

We are going to setup StandardRB as part of our development environment. This will involve some additional documentation so developers know how to integrate the tool into their text editors.
https://github.com/testdouble/standard#how-do-i-run-standard-in-my-editor

Before merging this we should come to a basic agreement on configuration and local setup.

• Setup standard https://github.com/testdouble/standard
• Add documentation for developers to integrate with text editor
• In separate commits, run the autoformatter to autofix any potential issues.
• In separate commits, fix individual lint violations that cannot be autocorrected. Flag a steward if you have questions about how to go about this.
• Ensure tests run and that a click through of the app, both admin and otherwise, still work as expected.

[rgenchev]

Can I work on this? Thanks in advance! :)

[midhunkrishna]

@kalimar @rgenchev,

I think the following plan of action should work:

1. Install rubocop like, in development and test groups

gem "rubocop-github"
gem "rubocop-performance"
gem "rubocop-rails"

2. Generate config by:

rubocop --auto-gen-config

This should create a .rubocop.yml file and .rubocop-todo.yml, which can be split into multiple tickets and worked on later without creating a single large PR (s ince this project is small and in the early stages, we can still go ahead and do an auto fix, wdyt @kalimar )

3. I would say the default configuration provided would suffice?

4. Editor specific setup instructions are provided in the rubocop documentation itself (https://rubocop.readthedocs.io/en/latest/integration_with_other_tools/#editor-integration). We can include this link to the readme.

[midhunkrishna]

Shouldn't we also have a section in here about the different tools that we use for dev?

Eg:

We Use:

• rubocop for ruby linting and for ruby styleguide
• jest as a js test runner
• simplecov for code coverage metric

@kalimar @MxOliver @mirandawang

[rgenchev]

I created #340. I wasn't sure if I should update the readme.

[midhunkrishna]

@rgenchev I wasn't sure about the readme either. You can update readme like this in this PR -> https://github.com/Terrastories/terrastories/pull/338/files#diff-550856cf67eb7260aaeb883550043af8R143-R149

[rgenchev]

@midhunkrishna, I did it here: 2a2d32c

[lauramosher]

Thoughts on utilizing StandardRB? It is essentially a wrapper around RuboCop, but defines "standards" for which rules are enforced rather than having to determine which ones we want to and why for each cop.

[rgenchev]

I could continue working on this. We just have to decide whether we pick rubocop or StandardRB.

[rgenchev]

@lauramosher, @midhunkrishna, @kalimar, what do you think? :)

[lauramosher]

@rgenchev HI! Thank you for your interest. I've updated the issue to reflect the decision the stewards have landed on for our linter and formatter. Let us know if you have any questions.

I've assigned you this issue. If you've changed your mind, please let us know and we'll unassign you. Thanks!