Skip to content

Update MultiplayerGameLobby controls visibility logic. #982

Update MultiplayerGameLobby controls visibility logic.

Update MultiplayerGameLobby controls visibility logic. #982

Triggered via pull request February 26, 2024 12:43
Status Failure
Total duration 2m 16s
Artifacts

build.yml

on: pull_request
Matrix: build-clients
Fit to window
Zoom out
Zoom in

Annotations

7 errors and 30 warnings
build-clients (Ares): DXMainClient/DXGUI/Multiplayer/GameLobby/MultiplayerGameLobby.cs#L12
The type or namespace name 'Localization' could not be found (are you missing a using directive or an assembly reference?)
build-clients (Ares)
Process completed with exit code 1.
build-clients (TS): DXMainClient/DXGUI/Multiplayer/GameLobby/MultiplayerGameLobby.cs#L12
The type or namespace name 'Localization' could not be found (are you missing a using directive or an assembly reference?)
build-clients (TS)
Process completed with exit code 1.
build-clients (YR)
The job was canceled because "Ares" failed.
build-clients (YR): DXMainClient/DXGUI/Multiplayer/GameLobby/MultiplayerGameLobby.cs#L12
The type or namespace name 'Localization' could not be found (are you missing a using directive or an assembly reference?)
build-clients (YR)
Process completed with exit code 1.
build-clients (Ares): ClientCore/OSVersion.cs#L1
Declare types in namespaces (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1050)
build-clients (Ares): ClientCore/Enums/AllowPrivateMessagesFromEnum.cs#L3
Rename type name AllowPrivateMessagesFromEnum so that it does not end in 'Enum' (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1711)
build-clients (Ares): ClientCore/SavedGameManager.cs#L15
Member 'saveRenameInProgress' is explicitly initialized to its default value (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1805)
build-clients (Ares): ClientCore/CnCNet5/GameCollection.cs#L15
Rename type name GameCollection so that it does not end in 'Collection' (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1711)
build-clients (Ares): ClientCore/CnCNet5/NameValidator.cs#L24
Prefer 'AsSpan' over 'Substring' when span-based overloads are available (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1846)
build-clients (Ares): ClientCore/ClientConfiguration.cs#L239
The behavior of 'int.ToString()' could vary based on the current user's locale settings. Replace this call in 'ClientConfiguration.GetThemeInfoFromIndex(int)' with a call to 'int.ToString(IFormatProvider)'. (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1305)
build-clients (Ares): ClientCore/ClientConfiguration.cs#L289
Prefer using 'string.Equals(string, StringComparison)' to perform a case-insensitive comparison, but keep in mind that this might cause subtle changes in behavior, so make sure to conduct thorough testing after applying the suggestion, or if culturally sensitive comparison is not required, consider using 'StringComparison.OrdinalIgnoreCase' (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1862)
build-clients (Ares): ClientCore/ClientConfiguration.cs#L295
Prefer using 'string.Equals(string, StringComparison)' to perform a case-insensitive comparison, but keep in mind that this might cause subtle changes in behavior, so make sure to conduct thorough testing after applying the suggestion, or if culturally sensitive comparison is not required, consider using 'StringComparison.OrdinalIgnoreCase' (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1862)
build-clients (Ares): ClientCore/CnCNet5/GameCollection.cs#L280
Use 'string.StartsWith(char)' instead of 'string.StartsWith(string)' when you have a string with a single char (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1866)
build-clients (Ares): ClientCore/CnCNet5/GameCollection.cs#L280
The behavior of 'string.StartsWith(string)' could vary based on the current user's locale settings. Replace this call in 'ClientCore.CnCNet5.GameCollection.GetIRCChannelNameFromIniFile(Rampastring.Tools.IniFile, string, string)' with a call to 'string.StartsWith(string, System.StringComparison)'. (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1310)
build-clients (TS): ClientCore/OSVersion.cs#L1
Declare types in namespaces (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1050)
build-clients (TS): ClientCore/Enums/AllowPrivateMessagesFromEnum.cs#L3
Rename type name AllowPrivateMessagesFromEnum so that it does not end in 'Enum' (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1711)
build-clients (TS): ClientCore/CnCNet5/GameCollection.cs#L15
Rename type name GameCollection so that it does not end in 'Collection' (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1711)
build-clients (TS): ClientCore/SavedGameManager.cs#L15
Member 'saveRenameInProgress' is explicitly initialized to its default value (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1805)
build-clients (TS): ClientCore/CnCNet5/NameValidator.cs#L24
Prefer 'AsSpan' over 'Substring' when span-based overloads are available (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1846)
build-clients (TS): ClientCore/I18N/Translation.cs#L91
Use 'ArgumentNullException.ThrowIfNull' instead of explicitly throwing a new exception instance (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1510)
build-clients (TS): ClientCore/INIProcessing/IniPreprocessInfoStore.cs#L54
Prefer 'static readonly' fields over constant array arguments if the called method is called repeatedly and is not mutating the passed array (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1861)
build-clients (TS): ClientCore/INIProcessing/IniPreprocessor.cs#L34
Member 'ProcessSection' does not access instance data and can be marked as static (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1822)
build-clients (TS): ClientCore/INIProcessing/IniPreprocessInfoStore.cs#L121
The behavior of 'int.ToString()' could vary based on the current user's locale settings. Replace this call in 'IniPreprocessInfoStore.Write()' with a call to 'int.ToString(IFormatProvider)'. (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1305)
build-clients (TS): ClientCore/ProfanityFilter.cs#L37
Use nameof in place of string literal 'censoredWords' (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1507)
build-clients (YR): ClientCore/OSVersion.cs#L1
Declare types in namespaces (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1050)
build-clients (YR): ClientCore/Enums/AllowPrivateMessagesFromEnum.cs#L3
Rename type name AllowPrivateMessagesFromEnum so that it does not end in 'Enum' (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1711)
build-clients (YR): ClientCore/SavedGameManager.cs#L15
Member 'saveRenameInProgress' is explicitly initialized to its default value (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1805)
build-clients (YR): ClientCore/CnCNet5/GameCollection.cs#L15
Rename type name GameCollection so that it does not end in 'Collection' (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1711)
build-clients (YR): ClientCore/CnCNet5/NameValidator.cs#L24
Prefer 'AsSpan' over 'Substring' when span-based overloads are available (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1846)
build-clients (YR): ClientCore/ClientConfiguration.cs#L239
The behavior of 'int.ToString()' could vary based on the current user's locale settings. Replace this call in 'ClientConfiguration.GetThemeInfoFromIndex(int)' with a call to 'int.ToString(IFormatProvider)'. (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1305)
build-clients (YR): ClientCore/CnCNet5/GameCollection.cs#L280
Use 'string.StartsWith(char)' instead of 'string.StartsWith(string)' when you have a string with a single char (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1866)
build-clients (YR): ClientCore/CnCNet5/GameCollection.cs#L297
Prefer using 'string.Equals(string, StringComparison)' to perform a case-insensitive comparison, but keep in mind that this might cause subtle changes in behavior, so make sure to conduct thorough testing after applying the suggestion, or if culturally sensitive comparison is not required, consider using 'StringComparison.OrdinalIgnoreCase' (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1862)
build-clients (YR): ClientCore/CnCNet5/GameCollection.cs#L280
The behavior of 'string.StartsWith(string)' could vary based on the current user's locale settings. Replace this call in 'ClientCore.CnCNet5.GameCollection.GetIRCChannelNameFromIniFile(Rampastring.Tools.IniFile, string, string)' with a call to 'string.StartsWith(string, System.StringComparison)'. (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1310)
build-clients (YR): ClientCore/CnCNet5/GameCollection.cs#L313
Prefer using 'string.Equals(string, StringComparison)' to perform a case-insensitive comparison, but keep in mind that this might cause subtle changes in behavior, so make sure to conduct thorough testing after applying the suggestion, or if culturally sensitive comparison is not required, consider using 'StringComparison.OrdinalIgnoreCase' (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1862)