-
Notifications
You must be signed in to change notification settings - Fork 73
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
Fixes #24734: Remove System compliance from Score #5605
Fixes #24734: Remove System compliance from Score #5605
Conversation
cp = complianceWithoutSystem.compliance.computePercent() | ||
event = ComplianceScoreEvent(nodeId, cp) | ||
_ <- scoreServiceManager.handleEvent(event) | ||
} yield () |
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.
since it's exactly the same code at two places and it's becoming non trivial, can you factor-it out somewhere ?
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.
Done!
Commit modified |
4494b6d
to
3ed979d
Compare
Commit modified |
3ed979d
to
c3b3b57
Compare
import net.liftweb.common.Box | ||
import net.liftweb.common.Full | ||
import org.joda.time.DateTime | ||
import org.junit.runner.RunWith | ||
import org.specs2.mutable.* | ||
import org.specs2.mutable._ |
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.
Seems like your intellij doesn't like scala 3 syntax :)
Commit modified |
c3b3b57
to
c41676e
Compare
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.
LGTM
OK, merging this PR |
9734452
into
Normation:branches/rudder/8.1
https://issues.rudder.io/issues/24734