Skip to content

add debt to application record#5

Open
mikesive wants to merge 1 commit intomasterfrom
add_debt
Open

add debt to application record#5
mikesive wants to merge 1 commit intomasterfrom
add_debt

Conversation

@mikesive
Copy link

Here is a PR with lots of debt.

class ApplicationRecord < ActiveRecord::Base
self.abstract_class = true

def set_bad_method
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Bliss.AI: report

Debt Issue
8.0 Perceived complexity for set_bad_method is too high. [19/7]
7.5 Cyclomatic complexity for set_bad_method is too high. [17/6]
3.1 Cyclomatic Complexity is too high for set_bad_method
2.0 Method has too many lines. [30/10]
0.5 Assignment Branch Condition size for set_bad_method is too high. [19.6/15]

@blissbot
Copy link
Collaborator

Bliss.AI: summary

Verdict Criteria
- No test coverage report
👎 0% test added, below acceptable levels
👍 No high churn files touched
+21.1 Complexity increased overall ⚠️
+12.0 Duplication increased overall ⚠️
+9.15 Style violations increased overall ⚠️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants