Boy scout rule. #345
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/Game.Actions.cs#L6
Refactor this method to add validation of parameter 'start' before using it.
|
|
build:
example/ConquerClub.Domain/Dice.cs#L22
The Cyclomatic Complexity of this method is 12 which is greater than 10 authorized.
|
|
build:
test/Qowaiv.DomainModel.UnitTests/Reflection/Expression_compiling_event_dispatcher_specs.cs#L93
Remove the unused internal class 'WhenWithDummyEvent'.
|
|
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.
|