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
refactor: move fields logic to concern #915
Conversation
return matched_field[:class] if matched_field.present? && matched_field[:class].present? | ||
end | ||
|
||
def parse_symbol(field_name, as:, **args, &block) |
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.
Method parse_symbol
has a Cognitive Complexity of 6 (exceeds 5 allowed). Consider refactoring.
Code Climate has analyzed commit a1f1b61 and detected 1 issue on this pull request. Here's the issue category breakdown:
View more on Code Climate. |
return matched_field[:class] if matched_field.present? && matched_field[:class].present? | ||
end | ||
|
||
def parse_symbol(field_name, as:, **args, &block) |
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.
Method parse_symbol
has a Cognitive Complexity of 7 (exceeds 5 allowed). Consider refactoring.
Codecov Report
@@ Coverage Diff @@
## main #915 +/- ##
==========================================
- Coverage 93.88% 93.86% -0.02%
==========================================
Files 483 483
Lines 9038 9050 +12
==========================================
+ Hits 8485 8495 +10
- Misses 553 555 +2
Continue to review full report at Codecov.
|
Description
Some house-cleaning. We're moving some things to concerns.
Checklist: