Skip to content

.NET standard interfaces#41

Merged
Seramis merged 1 commit intoByteAether:mainfrom
Seramis:dotnet-standard-interfaces
Apr 27, 2026
Merged

.NET standard interfaces#41
Seramis merged 1 commit intoByteAether:mainfrom
Seramis:dotnet-standard-interfaces

Conversation

@Seramis
Copy link
Copy Markdown
Member

@Seramis Seramis commented Apr 27, 2026

Description

Introduced relevant .NET standard interfaces. Replaced Max with MaxValue and introduced MinValue for as required by standard IMinMaxValue<T> interface. Marked Max as obsolete. Synced README and PACKAGE files with updates. Added and adjusted unit tests.

Type of Change

  • Bug fix (non-breaking change that fixes an issue)
  • New feature (non-breaking change that adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation update
  • Other (please specify):

Checklist

  • The PR is submitted to the correct branch (main).
  • My code follows the project's coding style. (.editorconfig)
  • I have commented my code, particularly in hard-to-understand areas and public interfaces.
  • I have added or updated tests for the changes I made.
  • All new and existing tests passed.
  • I have updated the documentation where applicable.

Additional Notes

ulid.Max has been marked as obsolete - ulid.MaxValue is the property that should be used based on .NET's IMinMaxValue<> interface. ulid.Empty is kept for semantic clarity.

…xValue` and introduced `MinValue` for as required by standard `IMinMaxValue<T>` interface. Marked `Max` as obsolete. Synced README and PACKAGE files with updates. Added and adjusted unit tests.
@Seramis Seramis self-assigned this Apr 27, 2026
@Seramis Seramis merged commit 396e59e into ByteAether:main Apr 27, 2026
5 checks passed
@Seramis Seramis deleted the dotnet-standard-interfaces branch April 27, 2026 10:43
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.

1 participant