Skip to content

EnumerateHours#16

Merged
Demosfen merged 4 commits into
mainfrom
EnumerateHours
Jan 29, 2026
Merged

EnumerateHours#16
Demosfen merged 4 commits into
mainfrom
EnumerateHours

Conversation

@Demosfen

Copy link
Copy Markdown
Member

No description provided.

@Demosfen Demosfen requested a review from alex1ozr January 29, 2026 13:45
@Demosfen Demosfen self-assigned this Jan 29, 2026
alex1ozr
alex1ozr previously approved these changes Jan 29, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This pull request adds a new EnumerateHours extension method for DateTimeRange and EnumerateHoursTo for DateTime, enabling enumeration of hours within a date/time range. The PR also includes code style improvements by converting several methods to use expression-bodied members.

Changes:

  • Added EnumerateHours and EnumerateHoursTo methods to enumerate hours in a date/time range
  • Refactored existing methods to use expression-bodied members and conditional throw expressions
  • Added unit test for the new EnumerateHours functionality
  • Bumped package version from 10.0.1 to 10.0.2

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 2 comments.

File Description
src/Utilities/System/DateTimeExtensions.cs Added EnumerateHoursTo method and refactored multiple methods to use expression-bodied syntax
src/Utilities/System/DateTimeRangeExtensions.cs Added EnumerateHours method and refactored EnumerateMinutes to use expression-bodied syntax
tests/UnitTests/Utilities/DateTimeRangeExtensionsTests.cs Added unit test for EnumerateHours_ReturnsAllHours
Directory.Build.props Incremented package version to 10.0.2 for the new feature

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread src/Utilities/System/DateTimeRangeExtensions.cs Outdated
Comment thread src/Utilities/System/DateTimeExtensions.cs
@Demosfen Demosfen merged commit 250ba4d into main Jan 29, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants