Skip to content

Add Code Climate and Rubocop configs#6

Merged
amoose merged 1 commit intomasterfrom
add-codeclimate
Apr 12, 2016
Merged

Add Code Climate and Rubocop configs#6
amoose merged 1 commit intomasterfrom
add-codeclimate

Conversation

@monfresh
Copy link
Contributor

Why: To enable static code analysis and ensure our code stays clean and secure

How: Add a .codeclimate.yml that configures the list of engines we want to use, and a .rubocop.yml to configure our Rubocop preferences

More info here: https://docs.codeclimate.com/docs/configuring-your-code-climate-analysis

**Why**: To enable static code analysis and ensure our code stays clean and secure

**How**: Add a .codeclimate.yml that configures the list of engines we want to use, and a .rubocop.yml to configure our Rubocop preferences

More info here: https://docs.codeclimate.com/docs/configuring-your-code-climate-analysis
@amoose
Copy link
Contributor

amoose commented Apr 12, 2016

@monfresh does codeclimate support more than one application per repository?

@amoose
Copy link
Contributor

amoose commented Apr 12, 2016

I want to pull this in to rebase my README update 🚀

@amoose amoose merged commit d68d7f5 into master Apr 12, 2016
@amoose amoose deleted the add-codeclimate branch April 12, 2016 19:57
@monfresh
Copy link
Contributor Author

Good question. I'm not sure.

@monfresh monfresh mentioned this pull request Apr 13, 2016
aduth added a commit that referenced this pull request Oct 14, 2021
**Why**: Because it's no longer used. It was added in #6 as part of CodeClimate checks, which was later removed in #3825.
aduth added a commit that referenced this pull request Oct 15, 2021
**Why**: Because it's no longer used. It was added in #6 as part of CodeClimate checks, which was later removed in #3825.
zachmargolis pushed a commit that referenced this pull request Sep 20, 2024
**Why**: It's quite possible for get_params to be defined
but lack a Signature.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants