-
Notifications
You must be signed in to change notification settings - Fork 2
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Rubocop applications #365
Rubocop applications #365
Conversation
@@ -16,40 +14,30 @@ | |||
|
|||
module PreAssembly | |||
class Smpl | |||
include PreAssembly::Logging |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Note: wasn't actually using Logging
!
@@ -101,7 +100,7 @@ def manifest_cols | |||
private | |||
|
|||
def normalize_dir(dir) | |||
dir.chomp('/') if dir | |||
dir&.chomp('/') |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've been waiting so long for (1) Ruby to get a safe accessor, and (2) for its use to become conventional. Woo!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yeah, i really like this too, it's a very nice usability enhancement.
Pull Request Test Coverage Report for Build 1147
💛 - Coveralls |
- accessors not needed, use readers instead - actually use the readers instead of instance vars in methods - remove `puts` statement and related `verbose` option - initialize `@manifest` in `#initialize` - some beautification
Excludes `StringLiterals` and `HashSyntax` as high churn for now.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ha, peter beat me to the merge. but 👍 from me too.
2 manual commits (Lint errors and Smpl.rb cleanup), otherwise all auto-corrections run by category.
Note this includes Performance and Security (yaml safe load) fixes.
I excluded
HashSyntax
andStringLiterals
which would touch another 350 lines. I also did not doRSpec
fixes, which tend to be more capricious.If this gets merged, I'll probably follow up with those and wire rubocop into CI.