Skip to content

Make more classes internal, reduce interface use #3554

Make more classes internal, reduce interface use

Make more classes internal, reduce interface use #3554

Triggered via pull request December 16, 2024 02:22
Status Failure
Total duration 1m 14s
Artifacts

test.yml

on: pull_request
test_and_check_coverage
27s
test_and_check_coverage
Matrix: test
Fit to window
Zoom out
Zoom in

Annotations

6 errors and 21 warnings
test_and_check_coverage: ImperatorToCK3.UnitTests/CK3/Characters/CK3CharacterTests.cs#L133
Inconsistent accessibility: parameter type 'ProvinceMapper' is less accessible than method 'CK3CharacterTests.CK3CharacterBuilder.WithProvinceMapper(ProvinceMapper)'
test_and_check_coverage
Process completed with exit code 1.
test (macos-14): ImperatorToCK3.UnitTests/CK3/Characters/CK3CharacterTests.cs#L133
Inconsistent accessibility: parameter type 'ProvinceMapper' is less accessible than method 'CK3CharacterTests.CK3CharacterBuilder.WithProvinceMapper(ProvinceMapper)'
test (macos-14)
Process completed with exit code 1.
test (self-hosted, windows)
The job was canceled because "macos-14" failed.
test (self-hosted, windows)
The operation was canceled.
test_and_check_coverage
Failed to install dotnet runtime + cli, exit code: 1. cp: cannot create directory '/usr/share/dotnet/host/fxr/8.0.11/': Permission denied cp: cannot create directory '/usr/share/dotnet/shared/Microsoft.NETCore.App/8.0.11/': Permission denied dotnet_install: Error: Failed to verify the version of installed `.NET Core Runtime`. Installation source: https://dotnetcli.azureedge.net/dotnet/Runtime/8.0.11/dotnet-runtime-8.0.11-linux-x64.tar.gz. Installation location: /usr/share/dotnet. Report the bug at https://github.com/dotnet/install-scripts/issues. dotnet_install: Error: `.NET Core Runtime` with version = 8.0.11 failed to install with an error.
test_and_check_coverage: ImperatorToCK3/CK3/CK3LocDB.cs#L93
TODO add unit test for combining loc from all the sources into one locblock (https://github.com/meziantou/Meziantou.Analyzer/blob/main/docs/Rules/MA0026.md)
test (macos-14): ImperatorToCK3/CK3/CK3LocDB.cs#L93
TODO add unit test for combining loc from all the sources into one locblock (https://github.com/meziantou/Meziantou.Analyzer/blob/main/docs/Rules/MA0026.md)