.NET 9 (#11)

johnkors · web-flow · commit a6b8d789156a · 2025-02-16T17:50:25.000+01:00
deprecate .NET 7
add .NET 9

+semver:major
diff --git a/.editorconfig b/.editorconfig
@@ -3,6 +3,7 @@ root = true
 
 [*]
 charset = utf-8
+insert_final_newline = true
 
 # C# files
 [*.cs]
@@ -15,8 +16,9 @@ indent_style = space
 tab_width = 4
 
 # New line preferences
-end_of_line = lf
-insert_final_newline = false
+insert_final_newline = true
+trim_trailing_whitespace = true
+
 
 #### .NET Coding Conventions ####
 
@@ -37,7 +39,7 @@ dotnet_style_parentheses_in_other_operators = never_if_unnecessary:silent
 dotnet_style_parentheses_in_relational_binary_operators = always_for_clarity:silent
 
 # Modifier preferences
-dotnet_style_require_accessibility_modifiers = for_non_interface_members:silent
+dotnet_style_require_accessibility_modifiers = for_non_interface_members:warning
 
 # Expression-level preferences
 csharp_style_deconstructed_variable_declaration = true:suggestion
@@ -65,9 +67,9 @@ dotnet_code_quality_unused_parameters = all:suggestion
 #### C# Coding Conventions ####
 
 # var preferences
-csharp_style_var_elsewhere = false:silent
-csharp_style_var_for_built_in_types = false:silent
-csharp_style_var_when_type_is_apparent = false:silent
+csharp_style_var_elsewhere = true:silent
+csharp_style_var_for_built_in_types = true:silent
+csharp_style_var_when_type_is_apparent = true:silent
 
 # Expression-bodied members
 csharp_style_expression_bodied_accessors = true:silent
@@ -87,7 +89,7 @@ csharp_style_pattern_matching_over_is_with_cast_check = true:suggestion
 csharp_style_conditional_delegate_call = true:suggestion
 
 # Modifier preferences
-csharp_preferred_modifier_order = public,private,protected,internal,static,extern,new,virtual,abstract,sealed,override,readonly,unsafe,volatile,async
+csharp_preferred_modifier_order = public, private, protected, internal, static, extern, new, virtual, abstract, sealed, override, readonly, unsafe, volatile, async
 
 # Code-block preferences
 csharp_prefer_braces = true:silent
@@ -100,6 +102,12 @@ csharp_style_prefer_range_operator = true:suggestion
 csharp_style_unused_value_assignment_preference = discard_variable:suggestion
 csharp_style_unused_value_expression_statement_preference = discard_variable:silent
 
+# C# 10
+csharp_style_namespace_declarations = file_scoped:error
+csharp_style_prefer_primary_constructors = true
+dotnet_diagnostic.IDE0290.severity = error
+
+
 #### C# Formatting Rules ####
 
 # New line preferences
@@ -146,3 +154,9 @@ csharp_space_between_square_brackets = false
 # Wrapping preferences
 csharp_preserve_single_line_blocks = true
 csharp_preserve_single_line_statements = true
+
+
+[*.csproj]
+indent_size = 2
+indent_style = space
+tab_width = 2
diff --git a/.github/workflows/CI.yml b/.github/workflows/CI.yml
@@ -15,10 +15,10 @@ jobs:
       matrix:
         os: [ubuntu-latest, windows-latest]
     steps:
-      - uses: actions/checkout@v3
-      - uses: actions/setup-dotnet@v3
+      - uses: actions/checkout@v4
+      - uses: actions/setup-dotnet@v4
         with:
-          dotnet-version: 8.0.x
+          dotnet-version: 9.0.x
       - run: dotnet --info
       - name: Restore dependencies
         run: dotnet restore
diff --git a/.github/workflows/PreRelease.yml b/.github/workflows/PreRelease.yml
@@ -8,12 +8,12 @@ jobs:
     runs-on: ubuntu-latest
 
     steps:
-      - uses: actions/checkout@v3
+      - uses: actions/checkout@v4
         with:
           fetch-depth: 0
-      - uses: actions/setup-dotnet@v3
+      - uses: actions/setup-dotnet@v4
         with:
-          dotnet-version: 8.0.x
+          dotnet-version: 9.0.x
       - name: Restore dependencies
         run: dotnet restore
       - name: Build
diff --git a/.github/workflows/Release.yml b/.github/workflows/Release.yml
@@ -11,9 +11,9 @@ jobs:
       - uses: actions/checkout@v3
         with:
           fetch-depth: 0
-      - uses: actions/setup-dotnet@v3
+      - uses: actions/setup-dotnet@v4
         with:
-          dotnet-version: 8.0.x
+          dotnet-version: 9.0.x
       - name: Restore dependencies
         run: dotnet restore
       - name: Build
diff --git a/src/CronBackgroundServices/CronBackgroundService.cs b/src/CronBackgroundServices/CronBackgroundService.cs
@@ -3,25 +3,16 @@
 
 namespace CronBackgroundServices;
 
-internal class CronBackgroundService : BackgroundService
+internal class CronBackgroundService(IRecurringAction Action, ILogger logger) : BackgroundService
 {
-    protected readonly IRecurringAction Action;
-    private readonly ILogger _logger;
-    private readonly Timing _timing;
+    private readonly Timing _timing = new(Action.GetTimeZoneId());
 
-    public CronBackgroundService(IRecurringAction action, ILogger logger)
-    {
-        _timing = new Timing(action.GetTimeZoneId());
-        Action = action;
-        _logger = logger;
-        Cron = action.Cron;
-        _logger.LogTrace($"Using {Cron} and timezone '{_timing.TimeZoneInfo.Id}. The time in this timezone: {_timing.RelativeNow()}'");
-    }
-
-    private string Cron { get; }
+    private string Cron { get; } = Action.Cron;
 
     protected override async Task ExecuteAsync(CancellationToken stoppingToken)
     {
+        logger.LogTrace(
+            $"Using {Cron} and timezone '{_timing.TimeZoneInfo.Id}. The time in this timezone: {_timing.RelativeNow()}'");
         DateTimeOffset? next = null;
 
         do
@@ -33,7 +24,7 @@ protected override async Task ExecuteAsync(CancellationToken stoppingToken)
                 next = _timing.GetNextOccurenceInRelativeTime(Cron);
                 var uText = _timing.Get10NextOccurrences(Cron);
                 var logText = $"Ten next occurrences :\n{uText.Aggregate((x, y) => x + "\n" + y)}";
-                _logger.LogTrace(logText);
+                logger.LogTrace(logText);
             }
 
             if (now > next)
@@ -44,11 +35,13 @@ protected override async Task ExecuteAsync(CancellationToken stoppingToken)
                 }
                 catch (Exception e)
                 {
-                    _logger.LogError(e, e.Message);
+                    logger.LogError(e, e.Message);
                 }
 
                 next = _timing.GetNextOccurenceInRelativeTime(Cron);
-                _logger.LogTrace($"Next at {next.Value.DateTime.ToLongDateString()} {next.Value.DateTime.ToLongTimeString()}");
+                logger.LogTrace(next is not null
+                    ? $"Next at {next.Value.DateTime.ToLongDateString()} {next.Value.DateTime.ToLongTimeString()}"
+                    : "No more occurences.");
             }
             else
             {
@@ -57,7 +50,6 @@ protected override async Task ExecuteAsync(CancellationToken stoppingToken)
                 // cron occurence (lowest possible: every second)
                 await Task.Delay(100);
             }
-
         } while (!stoppingToken.IsCancellationRequested);
     }
 }
diff --git a/src/CronBackgroundServices/CronBackgroundServices.csproj b/src/CronBackgroundServices/CronBackgroundServices.csproj
@@ -1,38 +1,33 @@
 ﻿<Project Sdk="Microsoft.NET.Sdk">
 
-    <PropertyGroup>
-        <TargetFrameworks>net8.0;net7.0</TargetFrameworks>
-        <RootNamespace>CronBackgroundServices</RootNamespace>
-        <PackageId>CronBackgroundServices</PackageId>
-        <Authors>John Korsnes</Authors>
-        <Description>
-            A .NET Core Background Service using Cron expressions as triggers
-        </Description>
-        <PackageTags>dotnetcore</PackageTags>
-        <PackageProjectUrl>https://github.com/slackbot-net/slackbot.net</PackageProjectUrl>
-        <License>https://github.com/slackbot-net/slackbot.net/blob/master/LICENSE</License>
-        <RepositoryUrl>https://github.com/slackbot-net/slackbot.net</RepositoryUrl>
-        <PackageIconUrl>images/cron.png</PackageIconUrl>
-        <PackageIcon>cron.png</PackageIcon>
-        <RepositoryType>git</RepositoryType>
-        <ImplicitUsings>enable</ImplicitUsings>
-        <LangVersion>latest</LangVersion>
-    </PropertyGroup>
-
-    <ItemGroup>
-        <PackageReference Include="Cronos" Version="0.7.1" />
-    </ItemGroup>
-    <ItemGroup Condition="'$(TargetFramework)' == 'net8.0'">
-        <PackageReference Include="Microsoft.Extensions.Hosting.Abstractions" Version="8.0.0" />
-        <PackageReference Include="Microsoft.Extensions.Logging.Abstractions" Version="8.0.0" />
-    </ItemGroup>
-    <ItemGroup Condition="'$(TargetFramework)' == 'net7.0'">
-        <PackageReference Include="Microsoft.Extensions.Hosting.Abstractions" Version="7.0.0" />
-        <PackageReference Include="Microsoft.Extensions.Logging.Abstractions" Version="7.0.0" />
-    </ItemGroup>
-
-    <ItemGroup>
-        <None Include="images/cron.png" Pack="true" PackagePath="" />
-    </ItemGroup>
+  <PropertyGroup>
+    <TargetFrameworks>net8.0;net9.0</TargetFrameworks>
+    <RootNamespace>CronBackgroundServices</RootNamespace>
+    <PackageId>CronBackgroundServices</PackageId>
+    <Authors>John Korsnes</Authors>
+    <Description>
+      A .NET Core Background Service using Cron expressions as triggers
+    </Description>
+    <PackageTags>dotnetcore</PackageTags>
+    <PackageProjectUrl>https://github.com/slackbot-net/slackbot.net</PackageProjectUrl>
+    <License>https://github.com/slackbot-net/slackbot.net/blob/master/LICENSE</License>
+    <RepositoryUrl>https://github.com/slackbot-net/slackbot.net</RepositoryUrl>
+    <PackageIconUrl>images/cron.png</PackageIconUrl>
+    <PackageIcon>cron.png</PackageIcon>
+    <RepositoryType>git</RepositoryType>
+    <ImplicitUsings>enable</ImplicitUsings>
+    <Nullable>enable</Nullable>
+    <LangVersion>latest</LangVersion>
+  </PropertyGroup>
 
+  <ItemGroup>
+    <PackageReference Include="Cronos" Version="0.7.1"/>
+  </ItemGroup>
+  <ItemGroup>
+    <PackageReference Include="Microsoft.Extensions.Hosting.Abstractions" Version="9.0.0"/>
+    <PackageReference Include="Microsoft.Extensions.Logging.Abstractions" Version="9.0.0"/>
+  </ItemGroup>
+  <ItemGroup>
+    <None Include="images/cron.png" Pack="true" PackagePath=""/>
+  </ItemGroup>
 </Project>
diff --git a/src/CronBackgroundServices/Hosting/ServiceCollectionExtensions.cs b/src/CronBackgroundServices/Hosting/ServiceCollectionExtensions.cs
@@ -4,10 +4,10 @@
 
 namespace CronBackgroundServices;
 
-public static class SlackbotWorkerBuilderExtensions
+public static class ServiceCollectionExtensions
 {
     /// <summary>
-    /// For distributed apps
+    ///     For distributed apps
     /// </summary>
     public static IServiceCollection AddRecurrer<T>(this IServiceCollection services) where T : class, IRecurringAction
     {
@@ -16,7 +16,7 @@ public static IServiceCollection AddRecurrer<T>(this IServiceCollection services
         {
             var allRecurrers = s.GetServices<IRecurringAction>();
             var single = allRecurrers.First(r => r is T);
-            var loggerFactory = s.GetService<ILoggerFactory>();
+            var loggerFactory = s.GetRequiredService<ILoggerFactory>();
             var logger = loggerFactory.CreateLogger<T>();
             return new CronBackgroundService(single, logger);
         });
diff --git a/src/CronBackgroundServices/IRecurringAction.cs b/src/CronBackgroundServices/IRecurringAction.cs
@@ -5,29 +5,28 @@ namespace CronBackgroundServices;
 public interface IRecurringAction
 {
     /// <summary>
-    /// The job to be executed at intervals defined by the Cron expression
+    ///     The cron expression (including seconds) as defined by the Cronos library:
+    ///     See https://github.com/HangfireIO/Cronos#cron-format
+    ///     Ex: Every second: */1 * * * * *
+    ///     Ex: Every minute: 0 */1 * * * *
+    ///     Ex: Every midnight: 0 0 */1 * * *
+    ///     Ex: First of every month 0 0 0 1 * *
     /// </summary>
-    /// <returns></returns>
-    Task Process(CancellationToken stoppingToken);
+    /// <returns>A valid Cron Expression</returns>
+    string Cron { get; }
 
     /// <summary>
-    /// The cron expression (including seconds) as defined by the Cronos library:
-    /// See https://github.com/HangfireIO/Cronos#cron-format
-    /// Ex: Every second: */1 * * * * *
-    /// Ex: Every minute: 0 */1 * * * *
-    /// Ex: Every midnight: 0 0 */1 * * *
-    /// Ex: First of every month 0 0 0 1 * *
+    ///     The job to be executed at intervals defined by the Cron expression
     /// </summary>
-    /// <returns>A valid Cron Expression</returns>
-    string Cron { get; }
+    /// <returns></returns>
+    Task Process(CancellationToken stoppingToken);
 
     /// <summary>
-    /// Optional: The TimeZone in which the Cron expression should be based on.
-    /// Defaults to UTC (Europe/London or GMT Standard Time)
-    ///
-    /// NB! When overriding this and targeting versions below .NET 6, use platform specific identifiers
-    /// If your runtime is .NET 6 or above, it's not required. It will handles the conversion:
-    /// See https://github.com/dotnet/runtime/pull/49412
+    ///     Optional: The TimeZone in which the Cron expression should be based on.
+    ///     Defaults to UTC (Europe/London or GMT Standard Time)
+    ///     NB! When overriding this and targeting versions below .NET 6, use platform specific identifiers
+    ///     If your runtime is .NET 6 or above, it's not required. It will handles the conversion:
+    ///     See https://github.com/dotnet/runtime/pull/49412
     /// </summary>
     /// <returns>timezoneId</returns>
     string GetTimeZoneId()
diff --git a/src/CronBackgroundServices/Timing.cs b/src/CronBackgroundServices/Timing.cs
@@ -2,14 +2,9 @@
 
 namespace CronBackgroundServices;
 
-internal class Timing
+internal class Timing(string timeZoneId)
 {
-    public readonly TimeZoneInfo TimeZoneInfo;
-
-    public Timing(string timeZoneId)
-    {
-        TimeZoneInfo = TimeZoneInfo.FindSystemTimeZoneById(timeZoneId);
-    }
+    public readonly TimeZoneInfo TimeZoneInfo = TimeZoneInfo.FindSystemTimeZoneById(timeZoneId);
 
     public DateTimeOffset RelativeNow(DateTimeOffset? nowutc = null)
     {
@@ -27,11 +22,13 @@ public IEnumerable<string> Get10NextOccurrences(string cron)
         var expression = CronExpression.Parse(cron, CronFormat.IncludeSeconds);
         var fromUtc = DateTime.UtcNow;
         var upcoming = new List<DateTime>();
-        upcoming.AddRange(Get10Occurrences(upcoming, expression, fromUtc, fromUtc.AddMonths(1)));
+        upcoming.AddRange(GetTenNextsOccurrences(upcoming, expression, fromUtc, fromUtc.AddMonths(1)));
         return upcoming.Select(u => $"{u.ToLongDateString()} {u.ToLongTimeString()}");
     }
 
-    private IEnumerable<DateTime> Get10Occurrences(List<DateTime> upcoming, CronExpression expression, DateTime fromUtc, DateTime toUtc)
+    private IEnumerable<DateTime> GetTenNextsOccurrences(List<DateTime> upcoming, CronExpression expression,
+        DateTime fromUtc,
+        DateTime toUtc)
     {
         while (true)
         {
@@ -43,9 +40,10 @@ private IEnumerable<DateTime> Get10Occurrences(List<DateTime> upcoming, CronExpr
             {
                 continue;
             }
+
             break;
         }
-        return upcoming.Take(10);
 
+        return upcoming.Take(10);
     }
 }

Original file line number	Diff line number	Diff line change
`@@ -3,25 +3,16 @@`
`3`	`3`
`4`	`4`	`namespace CronBackgroundServices;`
`5`	`5`
`6`		`-internal class CronBackgroundService : BackgroundService`
	`6`	`+internal class CronBackgroundService(IRecurringAction Action, ILogger logger) : BackgroundService`
`7`	`7`	`{`
`8`		`- protected readonly IRecurringAction Action;`
`9`		`- private readonly ILogger _logger;`
`10`		`- private readonly Timing _timing;`
	`8`	`+ private readonly Timing _timing = new(Action.GetTimeZoneId());`
`11`	`9`
`12`		`- public CronBackgroundService(IRecurringAction action, ILogger logger)`
`13`		`- {`
`14`		`- _timing = new Timing(action.GetTimeZoneId());`
`15`		`- Action = action;`
`16`		`- _logger = logger;`
`17`		`- Cron = action.Cron;`
`18`		`- _logger.LogTrace($"Using {Cron} and timezone '{_timing.TimeZoneInfo.Id}. The time in this timezone: {_timing.RelativeNow()}'");`
`19`		`- }`
`20`		`-`
`21`		`- private string Cron { get; }`
	`10`	`+ private string Cron { get; } = Action.Cron;`
`22`	`11`
`23`	`12`	`protected override async Task ExecuteAsync(CancellationToken stoppingToken)`
`24`	`13`	`{`
	`14`	`+ logger.LogTrace(`
	`15`	`+ $"Using {Cron} and timezone '{_timing.TimeZoneInfo.Id}. The time in this timezone: {_timing.RelativeNow()}'");`
`25`	`16`	`DateTimeOffset? next = null;`
`26`	`17`
`27`	`18`	`do`
`@@ -33,7 +24,7 @@ protected override async Task ExecuteAsync(CancellationToken stoppingToken)`
`33`	`24`	`next = _timing.GetNextOccurenceInRelativeTime(Cron);`
`34`	`25`	`var uText = _timing.Get10NextOccurrences(Cron);`
`35`	`26`	`var logText = $"Ten next occurrences :\n{uText.Aggregate((x, y) => x + "\n" + y)}";`
`36`		`- _logger.LogTrace(logText);`
	`27`	`+ logger.LogTrace(logText);`
`37`	`28`	`}`
`38`	`29`
`39`	`30`	`if (now > next)`
`@@ -44,11 +35,13 @@ protected override async Task ExecuteAsync(CancellationToken stoppingToken)`
`44`	`35`	`}`
`45`	`36`	`catch (Exception e)`
`46`	`37`	`{`
`47`		`- _logger.LogError(e, e.Message);`
	`38`	`+ logger.LogError(e, e.Message);`
`48`	`39`	`}`
`49`	`40`
`50`	`41`	`next = _timing.GetNextOccurenceInRelativeTime(Cron);`
`51`		`- _logger.LogTrace($"Next at {next.Value.DateTime.ToLongDateString()} {next.Value.DateTime.ToLongTimeString()}");`
	`42`	`+ logger.LogTrace(next is not null`
	`43`	`+ ? $"Next at {next.Value.DateTime.ToLongDateString()} {next.Value.DateTime.ToLongTimeString()}"`
	`44`	`+ : "No more occurences.");`
`52`	`45`	`}`
`53`	`46`	`else`
`54`	`47`	`{`
`@@ -57,7 +50,6 @@ protected override async Task ExecuteAsync(CancellationToken stoppingToken)`
`57`	`50`	`// cron occurence (lowest possible: every second)`
`58`	`51`	`await Task.Delay(100);`
`59`	`52`	`}`
`60`		`-`
`61`	`53`	`} while (!stoppingToken.IsCancellationRequested);`
`62`	`54`	`}`
`63`	`55`	`}`
Original file line number	Diff line number	Diff line change
`@@ -4,10 +4,10 @@`
`4`	`4`
`5`	`5`	`namespace CronBackgroundServices;`
`6`	`6`
`7`		`-public static class SlackbotWorkerBuilderExtensions`
	`7`	`+public static class ServiceCollectionExtensions`
`8`	`8`	`{`
`9`	`9`	`/// <summary>`
`10`		`- /// For distributed apps`
	`10`	`+ /// For distributed apps`
`11`	`11`	`/// </summary>`
`12`	`12`	`public static IServiceCollection AddRecurrer<T>(this IServiceCollection services) where T : class, IRecurringAction`
`13`	`13`	`{`
`@@ -16,7 +16,7 @@ public static IServiceCollection AddRecurrer<T>(this IServiceCollection services`
`16`	`16`	`{`
`17`	`17`	`var allRecurrers = s.GetServices<IRecurringAction>();`
`18`	`18`	`var single = allRecurrers.First(r => r is T);`
`19`		`- var loggerFactory = s.GetService<ILoggerFactory>();`
	`19`	`+ var loggerFactory = s.GetRequiredService<ILoggerFactory>();`
`20`	`20`	`var logger = loggerFactory.CreateLogger<T>();`
`21`	`21`	`return new CronBackgroundService(single, logger);`
`22`	`22`	`});`
Original file line number	Diff line number	Diff line change
`@@ -2,14 +2,9 @@`
`2`	`2`
`3`	`3`	`namespace CronBackgroundServices;`
`4`	`4`
`5`		`-internal class Timing`
	`5`	`+internal class Timing(string timeZoneId)`
`6`	`6`	`{`
`7`		`- public readonly TimeZoneInfo TimeZoneInfo;`
`8`		`-`
`9`		`- public Timing(string timeZoneId)`
`10`		`- {`
`11`		`- TimeZoneInfo = TimeZoneInfo.FindSystemTimeZoneById(timeZoneId);`
`12`		`- }`
	`7`	`+ public readonly TimeZoneInfo TimeZoneInfo = TimeZoneInfo.FindSystemTimeZoneById(timeZoneId);`
`13`	`8`
`14`	`9`	`public DateTimeOffset RelativeNow(DateTimeOffset? nowutc = null)`
`15`	`10`	`{`
`@@ -27,11 +22,13 @@ public IEnumerable<string> Get10NextOccurrences(string cron)`
`27`	`22`	`var expression = CronExpression.Parse(cron, CronFormat.IncludeSeconds);`
`28`	`23`	`var fromUtc = DateTime.UtcNow;`
`29`	`24`	`var upcoming = new List<DateTime>();`
`30`		`- upcoming.AddRange(Get10Occurrences(upcoming, expression, fromUtc, fromUtc.AddMonths(1)));`
	`25`	`+ upcoming.AddRange(GetTenNextsOccurrences(upcoming, expression, fromUtc, fromUtc.AddMonths(1)));`
`31`	`26`	`return upcoming.Select(u => $"{u.ToLongDateString()} {u.ToLongTimeString()}");`
`32`	`27`	`}`
`33`	`28`
`34`		`- private IEnumerable<DateTime> Get10Occurrences(List<DateTime> upcoming, CronExpression expression, DateTime fromUtc, DateTime toUtc)`
	`29`	`+ private IEnumerable<DateTime> GetTenNextsOccurrences(List<DateTime> upcoming, CronExpression expression,`
	`30`	`+ DateTime fromUtc,`
	`31`	`+ DateTime toUtc)`
`35`	`32`	`{`
`36`	`33`	`while (true)`
`37`	`34`	`{`
`@@ -43,9 +40,10 @@ private IEnumerable<DateTime> Get10Occurrences(List<DateTime> upcoming, CronExpr`
`43`	`40`	`{`
`44`	`41`	`continue;`
`45`	`42`	`}`
	`43`	`+`
`46`	`44`	`break;`
`47`	`45`	`}`
`48`		`- return upcoming.Take(10);`
`49`	`46`
	`47`	`+ return upcoming.Take(10);`
`50`	`48`	`}`
`51`	`49`	`}`