diff --git a/.editorconfig b/.editorconfig index 55dcd648..38a28be4 100644 --- a/.editorconfig +++ b/.editorconfig @@ -5,15 +5,8 @@ trim_trailing_whitespace = true insert_final_newline = true indent_style = space indent_size = 4 -end_of_line = lf [*.{csproj,json,config,yml,props}] indent_size = 2 -[*.sh] -end_of_line = lf - -[*.{cmd, bat}] -end_of_line = crlf - csharp_style_namespace_declarations = file_scoped:suggestion diff --git a/Build.ps1 b/Build.ps1 index e7982849..33920c45 100644 --- a/Build.ps1 +++ b/Build.ps1 @@ -7,7 +7,7 @@ Write-Output "build: Build started" Push-Location $PSScriptRoot try { - if(Test-Path .\artifacts) { + if (Test-Path .\artifacts) { Write-Output "build: Cleaning ./artifacts" Remove-Item ./artifacts -Force -Recurse } @@ -28,8 +28,9 @@ try { Write-Output "build: Package version suffix is $suffix" Write-Output "build: Build version suffix is $buildSuffix" + & dotnet format --no-restore --verify-no-changes --severity error & dotnet build -c Release --version-suffix=$buildSuffix /p:ContinuousIntegrationBuild=true - if($LASTEXITCODE -ne 0) { throw "Build failed" } + if ($LASTEXITCODE -ne 0) { throw "Build failed" } foreach ($src in Get-ChildItem src/*) { Push-Location $src @@ -41,7 +42,7 @@ try { } else { & dotnet pack -c Release --no-build --no-restore -o ../../artifacts } - if($LASTEXITCODE -ne 0) { throw "Packaging failed" } + if ($LASTEXITCODE -ne 0) { throw "Packaging failed" } Pop-Location } @@ -52,7 +53,7 @@ try { Write-Output "build: Testing project in $test" & dotnet test -c Release --no-build --no-restore - if($LASTEXITCODE -ne 0) { throw "Testing failed" } + if ($LASTEXITCODE -ne 0) { throw "Testing failed" } Pop-Location } @@ -65,7 +66,7 @@ try { foreach ($nupkg in Get-ChildItem artifacts/*.nupkg) { & dotnet nuget push -k $env:NUGET_API_KEY -s https://api.nuget.org/v3/index.json "$nupkg" - if($LASTEXITCODE -ne 0) { throw "Publishing failed" } + if ($LASTEXITCODE -ne 0) { throw "Publishing failed" } } if (!($suffix)) { diff --git a/src/Serilog.Settings.Configuration/ConfigurationLoggerConfigurationExtensions.cs b/src/Serilog.Settings.Configuration/ConfigurationLoggerConfigurationExtensions.cs index 0a2bb7d3..25f77ddc 100644 --- a/src/Serilog.Settings.Configuration/ConfigurationLoggerConfigurationExtensions.cs +++ b/src/Serilog.Settings.Configuration/ConfigurationLoggerConfigurationExtensions.cs @@ -218,8 +218,8 @@ public static LoggerConfiguration Configuration( { var configurationReader = readerOptions switch { - { ConfigurationAssemblySource: {} } => GetConfigurationReader(configuration, readerOptions, readerOptions.ConfigurationAssemblySource.Value), - { Assemblies: {} } => GetConfigurationReader(configuration, readerOptions, readerOptions.Assemblies), + { ConfigurationAssemblySource: { } } => GetConfigurationReader(configuration, readerOptions, readerOptions.ConfigurationAssemblySource.Value), + { Assemblies: { } } => GetConfigurationReader(configuration, readerOptions, readerOptions.Assemblies), _ => GetConfigurationReader(configuration, readerOptions ?? new ConfigurationReaderOptions(), readerOptions?.DependencyContext), }; return settingConfiguration.Settings(configurationReader); diff --git a/src/Serilog.Settings.Configuration/Properties/AssemblyInfo.cs b/src/Serilog.Settings.Configuration/Properties/AssemblyInfo.cs deleted file mode 100644 index 2527091f..00000000 --- a/src/Serilog.Settings.Configuration/Properties/AssemblyInfo.cs +++ /dev/null @@ -1,8 +0,0 @@ -using System.Runtime.CompilerServices; - -[assembly: InternalsVisibleTo("Serilog.Settings.Configuration.Tests, PublicKey=" + - "0024000004800000940000000602000000240000525341310004000001000100fb8d13fd344a1c" + - "6fe0fe83ef33c1080bf30690765bc6eb0df26ebfdf8f21670c64265b30db09f73a0dea5b3db4c9" + - "d18dbf6d5a25af5ce9016f281014d79dc3b4201ac646c451830fc7e61a2dfd633d34c39f87b818" + - "94191652df5ac63cc40c77f3542f702bda692e6e8a9158353df189007a49da0f3cfd55eb250066" + - "b19485ec")] \ No newline at end of file diff --git a/src/Serilog.Settings.Configuration/Serilog.Settings.Configuration.csproj b/src/Serilog.Settings.Configuration/Serilog.Settings.Configuration.csproj index 0b8d72a5..51e95896 100644 --- a/src/Serilog.Settings.Configuration/Serilog.Settings.Configuration.csproj +++ b/src/Serilog.Settings.Configuration/Serilog.Settings.Configuration.csproj @@ -1,4 +1,4 @@ - + Microsoft.Extensions.Configuration (appsettings.json) support for Serilog. @@ -6,7 +6,6 @@ net462;netstandard2.0;net8.0;net9.0 - true serilog;json;appsettings icon.png README.md @@ -16,15 +15,12 @@ Serilog - - - - - + + diff --git a/src/Serilog.Settings.Configuration/Settings/Configuration/ObjectArgumentValue.cs b/src/Serilog.Settings.Configuration/Settings/Configuration/ObjectArgumentValue.cs index be6e57e4..82a68a63 100644 --- a/src/Serilog.Settings.Configuration/Settings/Configuration/ObjectArgumentValue.cs +++ b/src/Serilog.Settings.Configuration/Settings/Configuration/ObjectArgumentValue.cs @@ -29,7 +29,7 @@ public ObjectArgumentValue(IConfigurationSection section, IReadOnlyCollection logger/sink config parameter? var typeInfo = toType.GetTypeInfo(); if (typeInfo.IsGenericType && - typeInfo.GetGenericTypeDefinition() is {} genericType && genericType == typeof(Action<>)) + typeInfo.GetGenericTypeDefinition() is { } genericType && genericType == typeof(Action<>)) { var configType = typeInfo.GenericTypeArguments[0]; IConfigurationReader configReader = new ConfigurationReader(_section, _configurationAssemblies, resolutionContext); @@ -185,14 +185,28 @@ bool TryCallCtor(Type type, Dictionary suppliedAr } else { - return new { ci, args, isCallable = false, matches, stringMatches, - usedArguments = suppliedNames }; + return new + { + ci, + args, + isCallable = false, + matches, + stringMatches, + usedArguments = suppliedNames + }; } } } - return new { ci, args, isCallable = true, matches, stringMatches, - usedArguments = suppliedNames }; + return new + { + ci, + args, + isCallable = true, + matches, + stringMatches, + usedArguments = suppliedNames + }; }) .Where(binding => binding.isCallable) .OrderByDescending(binding => binding.matches) diff --git a/test/Serilog.Settings.Configuration.Tests/ApiApprovalTests.cs b/test/Serilog.Settings.Configuration.Tests/Approval/ApiApprovalTests.cs similarity index 85% rename from test/Serilog.Settings.Configuration.Tests/ApiApprovalTests.cs rename to test/Serilog.Settings.Configuration.Tests/Approval/ApiApprovalTests.cs index 08bf7f46..fd50fc66 100644 --- a/test/Serilog.Settings.Configuration.Tests/ApiApprovalTests.cs +++ b/test/Serilog.Settings.Configuration.Tests/Approval/ApiApprovalTests.cs @@ -1,4 +1,4 @@ -#if NET7_0 +#if NET9_0 using PublicApiGenerator; using Shouldly; @@ -15,7 +15,7 @@ public void PublicApi_Should_Not_Change_Unintentionally() new() { IncludeAssemblyAttributes = false, - ExcludeAttributes = new[] { "System.Diagnostics.DebuggerDisplayAttribute" }, + ExcludeAttributes = [ "System.Diagnostics.DebuggerDisplayAttribute" ], }); publicApi.ShouldMatchApproved(options => options.WithFilenameGenerator((_, _, fileType, fileExtension) => $"{assembly.GetName().Name!}.{fileType}.{fileExtension}")); diff --git a/test/Serilog.Settings.Configuration.Tests/Serilog.Settings.Configuration.approved.txt b/test/Serilog.Settings.Configuration.Tests/Approval/Serilog.Settings.Configuration.approved.txt similarity index 100% rename from test/Serilog.Settings.Configuration.Tests/Serilog.Settings.Configuration.approved.txt rename to test/Serilog.Settings.Configuration.Tests/Approval/Serilog.Settings.Configuration.approved.txt diff --git a/test/Serilog.Settings.Configuration.Tests/Support/ConfigurationReaderTestHelpers.cs b/test/Serilog.Settings.Configuration.Tests/Support/ConfigurationReaderTestHelpers.cs index ae76b43f..f5e0af53 100644 --- a/test/Serilog.Settings.Configuration.Tests/Support/ConfigurationReaderTestHelpers.cs +++ b/test/Serilog.Settings.Configuration.Tests/Support/ConfigurationReaderTestHelpers.cs @@ -53,7 +53,7 @@ public static IConfigurationRoot GetConfigRoot( { var configBuilder = new ConfigurationBuilder(); - configBuilder.AddJsonString(appsettingsJsonLevel ?? "{}"); + configBuilder.AddJsonString(appsettingsJsonLevel ?? "{}"); configBuilder.AddJsonString(appsettingsDevelopmentJsonLevel ?? "{}"); configBuilder.Add(new ReloadableConfigurationSource(envVariables ?? new Dictionary())); diff --git a/test/TestDummies/TestDummies.csproj b/test/TestDummies/TestDummies.csproj index 25a7376f..43f8c6d8 100644 --- a/test/TestDummies/TestDummies.csproj +++ b/test/TestDummies/TestDummies.csproj @@ -14,8 +14,4 @@ - - - -