Clean-ups. #342
CornielAnnotations
11 warnings
|
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:
src/Qowaiv.DomainModel/EventDispatcher.cs#L4
Interface names should begin with I
|
|
build:
src/Qowaiv.DomainModel/EventDispatcher.cs#L4
Interface names should begin with I
|
|
build:
src/Qowaiv.DomainModel/EventDispatcher.cs#L4
Interface names should begin with I
|
|
build:
src/Qowaiv.DomainModel/EventDispatcher.cs#L4
Interface names should begin with I
|
|
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/Commands/CommandProcessor_specs.cs#L43
Cannot convert null literal to non-nullable reference type.
|
|
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#L83
Consider using the constraint model, Assert.That(actual, Is.EqualTo(expected)), instead of the classic model, Assert.AreEqual(expected, actual)
|