Skip to content

Merge pull request #40 from Pier-Two/dev #273

Merge pull request #40 from Pier-Two/dev

Merge pull request #40 from Pier-Two/dev #273

Triggered via push November 4, 2024 07:16
Status Success
Total duration 7m 32s
Artifacts
Build and analyze SonarCloud
5m 40s
Build and analyze SonarCloud
Fit to window
Zoom out
Zoom in

Annotations

22 warnings
Build and Test
The following actions use a deprecated Node.js version and will be forced to run on node20: actions/setup-dotnet@v3, actions/checkout@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 and Test: lib/SszSharp/SszSharp/BeaconState.cs#L5
Non-nullable property 'PreviousVersion' must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring the property as nullable.
Build and Test: lib/SszSharp/SszSharp/BeaconState.cs#L6
Non-nullable property 'CurrentVersion' must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring the property as nullable.
Build and Test: lib/SszSharp/SszSharp/BeaconState.cs#L21
Non-nullable property 'AttestingIndices' must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring the property as nullable.
Build and Test: lib/SszSharp/SszSharp/BeaconState.cs#L22
Non-nullable property 'Data' must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring the property as nullable.
Build and Test: lib/SszSharp/SszSharp/BeaconState.cs#L23
Non-nullable property 'Signature' must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring the property as nullable.
Build and Test: lib/SszSharp/SszSharp/BeaconState.cs#L14
Non-nullable property 'ParentRoot' must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring the property as nullable.
Build and Test: lib/SszSharp/SszSharp/BeaconState.cs#L15
Non-nullable property 'StateRoot' must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring the property as nullable.
Build and Test: lib/SszSharp/SszSharp/BeaconState.cs#L16
Non-nullable property 'BodyRoot' must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring the property as nullable.
Build and Test: lib/SszSharp/SszSharp/BeaconState.cs#L28
Non-nullable property 'AttestingIndices' must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring the property as nullable.
Build and Test: lib/SszSharp/SszSharp/BeaconState.cs#L29
Non-nullable property 'Data' must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring the property as nullable.
Build and analyze SonarCloud
The following actions use a deprecated Node.js version and will be forced to run on node20: actions/setup-dotnet@v3, actions/setup-java@v3, actions/checkout@v3, actions/cache@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 and analyze SonarCloud: lib/cortex/src/Cortex.Containers/BeaconBlock.cs#L23
Remove this commented out code. (https://rules.sonarsource.com/csharp/RSPEC-125)
Build and analyze SonarCloud: lib/cortex/src/Cortex.Containers/BeaconBlockBody.cs#L67
Remove this commented out code. (https://rules.sonarsource.com/csharp/RSPEC-125)
Build and analyze SonarCloud: lib/cortex/src/Cortex.Containers/BeaconState.cs#L13
Remove this commented out code. (https://rules.sonarsource.com/csharp/RSPEC-125)
Build and analyze SonarCloud: lib/cortex/src/Cortex.Containers/BeaconState.cs#L19
Remove this commented out code. (https://rules.sonarsource.com/csharp/RSPEC-125)
Build and analyze SonarCloud: lib/cortex/src/Cortex.Containers/BeaconState.cs#L93
Remove this commented out code. (https://rules.sonarsource.com/csharp/RSPEC-125)
Build and analyze SonarCloud: lib/cortex/src/Cortex.Containers/BeaconState.cs#L131
Remove this commented out code. (https://rules.sonarsource.com/csharp/RSPEC-125)
Build and analyze SonarCloud: lib/cortex/src/Cortex.Containers/BlsPublicKey.cs#L10
Remove this commented out code. (https://rules.sonarsource.com/csharp/RSPEC-125)
Build and analyze SonarCloud: lib/cortex/src/Cortex.Containers/Transfer.cs#L6
Remove this commented out code. (https://rules.sonarsource.com/csharp/RSPEC-125)
Build and analyze SonarCloud: lib/cortex/src/Cortex.Containers/Transfer.cs#L3
Remove this empty namespace. (https://rules.sonarsource.com/csharp/RSPEC-3261)
Build and analyze SonarCloud: lib/cortex/src/Cortex.Containers/AttestationData.cs#L5
Seal class 'AttestationData' or implement 'IEqualityComparer<T>' instead. (https://rules.sonarsource.com/csharp/RSPEC-4035)