feat: refactor TemplateEngine into focused single-responsibility clas… #118
iammukeshmci.yml
on: push
Build
1m 45s
Matrix: test
Publish Dev Containers
1m 37s
Publish Release (NuGet + Containers)
Annotations
14 warnings
|
Build:
src/Tools/CLI/Services/PackageUpdater.cs#L103
Modify 'UpdatePackagesPropsAsync' to catch a more specific allowed exception type, or rethrow the exception (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1031)
|
|
Build:
src/Tools/CLI/Services/PackageUpdater.cs#L187
Modify 'RestoreBackupAsync' to catch a more specific allowed exception type, or rethrow the exception (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1031)
|
|
Build:
src/Tools/CLI/Services/PackageUpdater.cs#L162
Modify 'CreateBackupAsync' to catch a more specific allowed exception type, or rethrow the exception (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1031)
|
|
Build:
src/Tools/CLI/Services/PackageUpdater.cs#L137
Modify 'UpdateManifestAsync' to catch a more specific allowed exception type, or rethrow the exception (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1031)
|
|
Build:
src/Tools/CLI/Services/VersionComparer.cs#L140
'string.IndexOf(char)' has a method overload that takes a 'StringComparison' parameter. Replace this call in 'FSH.CLI.Services.VersionComparer.ParseVersionParts(string)' with a call to 'string.IndexOf(char, System.StringComparison)' for clarity of intent. (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1307)
|
|
Build:
src/Tools/CLI/Services/VersionComparer.cs#L118
Remove this unused method parameter 'package'. (https://rules.sonarsource.com/csharp/RSPEC-1172)
|
|
Build:
src/Tools/CLI/Services/VersionComparer.cs#L34
Modify 'ParsePackagesProps' to catch a more specific allowed exception type, or rethrow the exception (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1031)
|
|
Build:
src/Tools/CLI/Services/VersionComparer.cs#L126
Merge this if statement with the enclosing one. (https://rules.sonarsource.com/csharp/RSPEC-1066)
|
|
Build:
src/Tools/CLI/Services/PackageUpdater.cs#L9
Add a 'private' constructor or the 'static' keyword to the class declaration. (https://rules.sonarsource.com/csharp/RSPEC-1118)
|
|
Build:
src/Tools/CLI/Scaffolding/TemplateRenderer.cs#L14
Remove this unread private field '_templateCache' or refactor the code to use its value. (https://rules.sonarsource.com/csharp/RSPEC-4487)
|
|
Publish Dev Containers:
src/Modules/Identity/Modules.Identity/Services/IdentityService.cs#L180
Member 'CreateBasicClaims' does not access instance data and can be marked as static (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1822)
|
|
Publish Dev Containers:
src/Modules/Identity/Modules.Identity/Services/IdentityService.cs#L180
Make 'CreateBasicClaims' a static method. (https://rules.sonarsource.com/csharp/RSPEC-2325)
|
|
Publish Dev Containers:
src/BuildingBlocks/Mailing/Services/SmtpMailService.cs#L49
Member 'ConfigureRecipients' does not access instance data and can be marked as static (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1822)
|
|
Publish Dev Containers:
src/BuildingBlocks/Mailing/Services/SmtpMailService.cs#L49
Make 'ConfigureRecipients' a static method. (https://rules.sonarsource.com/csharp/RSPEC-2325)
|
Artifacts
Produced during runtime
| Name | Size | Digest | |
|---|---|---|---|
|
build-output
Expired
|
139 MB |
sha256:2f0d0b8ff1dc19239979d17e56b20d2fe7f35be173dcd9615618c90cc8c92408
|
|