Skip to content

Add GitHub Action for Pull Requests #1

Add GitHub Action for Pull Requests

Add GitHub Action for Pull Requests #1

Triggered via pull request August 14, 2024 16:16
Status Success
Total duration 58s
Artifacts

pull_request_validation.yml

on: pull_request
Fit to window
Zoom out
Zoom in

Annotations

21 warnings
build
The following actions use a deprecated Node.js version and will be forced to run on node20: actions/checkout@v3, actions/setup-dotnet@v3. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
build: BedBrigade.Common/Common.cs#L433
Possible null reference return.
build: BedBrigade.Common/DateUtil.cs#L62
The variable 'ex' is declared but never used
build: BedBrigade.Common/Extensions.cs#L29
The variable 'ex' is declared but never used
build: BedBrigade.Common/EnumNameValue.cs#L5
Non-nullable property 'Value' must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring the property as nullable.
build: BedBrigade.Common/EnumNameValue.cs#L6
Non-nullable property 'Name' must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring the property as nullable.
build: BedBrigade.Common/EnumHelper.cs#L10
Dereference of a possibly null reference.
build: BedBrigade.Tests/ImageRotatorLogicTests.cs#L20
Consider using the constraint model, Assert.That(actual, Is.EqualTo(expected)), instead of the classic model, Assert.AreEqual(expected, actual) (https://github.com/nunit/nunit.analyzers/tree/master/documentation/NUnit2005.md)
build: BedBrigade.Tests/ImageRotatorLogicTests.cs#L36
Consider using the constraint model, Assert.That(actual, Is.EqualTo(expected)), instead of the classic model, Assert.AreEqual(expected, actual) (https://github.com/nunit/nunit.analyzers/tree/master/documentation/NUnit2005.md)
build: BedBrigade.Tests/ValidationTests.cs#L72
Consider using the constraint model, Assert.That(actual, Is.EqualTo(expected)), instead of the classic model, Assert.AreEqual(expected, actual) (https://github.com/nunit/nunit.analyzers/tree/master/documentation/NUnit2005.md)
build: BedBrigade.Tests/ImageRotatorLogicTests.cs#L68
Consider using the constraint model, Assert.That(actual, Is.EqualTo(expected)), instead of the classic model, Assert.AreEqual(expected, actual) (https://github.com/nunit/nunit.analyzers/tree/master/documentation/NUnit2005.md)
build: BedBrigade.Common/EnumHelper.cs#L10
Dereference of a possibly null reference.
build: BedBrigade.Common/EnumNameValue.cs#L5
Non-nullable property 'Value' must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring the property as nullable.
build: BedBrigade.Common/EnumNameValue.cs#L6
Non-nullable property 'Name' must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring the property as nullable.
build: BedBrigade.Common/DateUtil.cs#L62
The variable 'ex' is declared but never used
build: BedBrigade.Common/Extensions.cs#L29
The variable 'ex' is declared but never used
build: BedBrigade.Common/Common.cs#L433
Possible null reference return.
build: BedBrigade.Tests/ImageRotatorLogicTests.cs#L20
Consider using the constraint model, Assert.That(actual, Is.EqualTo(expected)), instead of the classic model, Assert.AreEqual(expected, actual) (https://github.com/nunit/nunit.analyzers/tree/master/documentation/NUnit2005.md)
build: BedBrigade.Tests/ImageRotatorLogicTests.cs#L36
Consider using the constraint model, Assert.That(actual, Is.EqualTo(expected)), instead of the classic model, Assert.AreEqual(expected, actual) (https://github.com/nunit/nunit.analyzers/tree/master/documentation/NUnit2005.md)
build: BedBrigade.Tests/ValidationTests.cs#L72
Consider using the constraint model, Assert.That(actual, Is.EqualTo(expected)), instead of the classic model, Assert.AreEqual(expected, actual) (https://github.com/nunit/nunit.analyzers/tree/master/documentation/NUnit2005.md)
build: BedBrigade.Tests/ImageRotatorLogicTests.cs#L52
Consider using the constraint model, Assert.That(actual, Is.EqualTo(expected)), instead of the classic model, Assert.AreEqual(expected, actual) (https://github.com/nunit/nunit.analyzers/tree/master/documentation/NUnit2005.md)