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
chore: Fix RuboCop Style/FormatStringToken #914
chore: Fix RuboCop Style/FormatStringToken #914
Conversation
.rubocop_todo.yml
Outdated
@@ -1,6 +1,6 @@ | |||
# This configuration was generated by | |||
# `rubocop --auto-gen-config` | |||
# on 2019-03-03 10:30:09 +0100 using RuboCop version 0.65.0. | |||
# on 2019-03-03 19:25:18 +0900 using RuboCop version 0.65.0. |
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.
Ah, one of the new tricks is to revert JUST this.
# on 2019-03-03 19:25:18 +0900 using RuboCop version 0.65.0. | |
# on 2019-03-03 10:30:09 +0100 using RuboCop version 0.65.0. |
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.
See the #854 under Workflow.
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.
Thank you for your advise!
Does that mean that the time(19:25:18 +0900) has reverted?
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.
Hey, awesome! We're getting closer!
I made some small work notes inline.
@@ -99,7 +99,8 @@ def self.decode(query) | |||
if !last_subkey || is_array | |||
value_type = is_array ? Array : Hash | |||
if context[subkey] && !context[subkey].is_a?(value_type) | |||
raise TypeError, format("expected %s (got %s) for param `%s'", value_type.name, context[subkey].class.name, subkey) | |||
raise TypeError, format("expected %<value_type_name>s (got %<class_name>s) for param `%<subkey>s'", | |||
value_type_name: value_type.name, class_name: context[subkey].class.name, subkey: subkey) |
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.
If this would go on more lines, each key-value pair in the hash on its own line, we could avoid adding new Metrics/LineLength issues.
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.
Thank you for your comments!
I try do it.
.rubocop_todo.yml
Outdated
@@ -1,6 +1,6 @@ | |||
# This configuration was generated by | |||
# `rubocop --auto-gen-config` | |||
# on 2019-03-03 10:30:09 +0100 using RuboCop version 0.65.0. | |||
# on 2019-03-03 19:25:18 +0900 using RuboCop version 0.65.0. |
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.
See the #854 under Workflow.
f673ac5
to
1058f1c
Compare
See #854 - this PR fixes Style/FormatStringToken