Boy-scout. #347
CornielAnnotations
1 error and 11 warnings
|
build
Process completed with exit code 1.
|
|
build
Node.js 12 actions are deprecated. Please update the following actions to use Node.js 16: actions/setup-dotnet@v1. For more information see: https://github.blog/changelog/2022-09-22-github-actions-all-actions-will-begin-running-on-node16-instead-of-node12/.
|
|
build:
example/ConquerClub.Domain/Events/Finished.cs#L3
Remove this empty record, write its code or make it an "interface".
|
|
build:
example/ConquerClub.Domain/Dice.cs#L22
The Cyclomatic Complexity of this method is 12 which is greater than 10 authorized.
|
|
build:
example/ConquerClub.Domain/Game.Actions.cs#L6
Refactor this method to add validation of parameter 'start' before using it.
|
|
build:
test/Qowaiv.DomainModel.UnitTests/TestTools/AggregateRootAssertTest.cs#L55
Consider using the constraint model, Assert.That(actual, Is.EqualTo(expected)), instead of the classic model, Assert.AreEqual(expected, actual)
|
|
build:
test/Qowaiv.DomainModel.UnitTests/TestTools/AggregateRootAssertTest.cs#L108
Consider using the constraint model, Assert.That(actual, Is.EqualTo(expected)), instead of the classic model, Assert.AreEqual(expected, actual)
|
|
build:
test/Qowaiv.DomainModel.UnitTests/TestTools/AggregateRootAssertTest.cs#L83
Consider using the constraint model, Assert.That(actual, Is.EqualTo(expected)), instead of the classic model, Assert.AreEqual(expected, actual)
|
|
build:
test/Qowaiv.DomainModel.UnitTests/TestTools/AggregateRootAssertTest.cs#L133
Consider using the constraint model, Assert.That(actual, Is.EqualTo(expected)), instead of the classic model, Assert.AreEqual(expected, actual)
|
|
build:
test/Qowaiv.DomainModel.UnitTests/TestTools/AggregateRootAssertTest.cs#L153
Consider using the constraint model, Assert.That(actual, Is.EqualTo(expected)), instead of the classic model, Assert.AreEqual(expected, actual)
|
|
build:
example/ConquerClub.UnitTests/Dice_specs.cs#L75
Refactor this code to not nest more than 3 control flow statements.
|
|
build:
example/ConquerClub.UnitTests/Dice_specs.cs#L122
Refactor this method to reduce its Cognitive Complexity from 16 to the 15 allowed.
|