Skip to content

Commit

Permalink
update codeclimate rules to match yii style
Browse files Browse the repository at this point in the history
follow up to yiisoft#13121
  • Loading branch information
cebe committed Dec 2, 2016
1 parent fc0752f commit faea888
Showing 1 changed file with 20 additions and 1 deletion.
21 changes: 20 additions & 1 deletion .codeclimate.yml
Original file line number Diff line number Diff line change
Expand Up @@ -12,11 +12,30 @@ engines:
enabled: true
phpmd:
enabled: true
# configure checks, see https://phpmd.org/rules/index.html for details
checks:
# Static access on Yii::$app is normal in Yii
CleanCode/StaticAccess:
enabled: false
# Yii is a framework so if fulfills the job of encapsulating superglobals
Controversial/Superglobals:
enabled: false
# allow private properties to start with $_
Controversial/CamelCasePropertyName:
enabled: false
enabled: true
allow-underscore: true
# Short variable names are no problem in most cases, e.g. $n = count(...);
Naming/ShortVariable:
enabled: false
# Long variable names can help with better understanding so we increase the limit a bit
Naming/LongVariable:
enabled: true
maximum: 25
# method names like up(), gc(), ... are okay.
Naming/ShortMethodName:
enabled: true
minimum: 2

ratings:
paths:
- "**.js"
Expand Down

0 comments on commit faea888

Please sign in to comment.