Skip to content

Release preparations 0.1.0 #20

Merged
feO2x merged 15 commits intomainfrom
19-release-preparations
Feb 24, 2026
Merged

Release preparations 0.1.0 #20
feO2x merged 15 commits intomainfrom
19-release-preparations

Conversation

@feO2x
Copy link
Copy Markdown
Owner

@feO2x feO2x commented Feb 24, 2026

Closes #19

  • adds reamde
  • adds publish on NuGet workflow
  • add package release notes

@feO2x feO2x self-assigned this Feb 24, 2026
@feO2x feO2x requested a review from Copilot February 24, 2026 08:13
@feO2x feO2x added documentation Improvements or additions to documentation enhancement New feature or request labels Feb 24, 2026
Copy link
Copy Markdown

Copilot AI left a comment

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 prepares the Light.Results library for its initial 0.1.0 release on NuGet. The library implements the Result Pattern with serialization support for HTTP and CloudEvents, including ASP.NET Core integrations.

Changes:

  • Added comprehensive README.md with documentation, examples, and configuration guides
  • Created GitHub Actions workflow for automated NuGet package publishing on release
  • Added package metadata (descriptions, release notes, repository URLs, and licensing) to all project files
  • Added Microsoft.SourceLink.GitHub for source debugging support
  • Included package icon and public signing key

Reviewed changes

Copilot reviewed 15 out of 17 changed files in this pull request and generated 6 comments.

Show a summary per file
File Description
README.md Comprehensive documentation with quickstart examples, usage patterns, and configuration options
.github/workflows/release-on-nuget.yml Automated workflow for building, signing, and publishing NuGet packages
Directory.Build.props Added version 0.1.0 for the release
src/Directory.Build.props Added NuGet package metadata including repository URLs, licensing, icon, and SourceLink configuration
Directory.Packages.props Added Microsoft.SourceLink.GitHub package version and reordered coverlet.collector
src/Light.Results/Light.Results.csproj Added package description and release notes for core library
src/Light.Results.AspNetCore.Shared/Light.Results.AspNetCore.Shared.csproj Added package description and release notes for shared ASP.NET Core functionality
src/Light.Results.AspNetCore.Mvc/Light.Results.AspNetCore.Mvc.csproj Added package description and release notes for MVC integration
src/Light.Results.AspNetCore.MinimalApis/Light.Results.AspNetCore.MinimalApis.csproj Added package description and release notes for Minimal APIs integration
src/Light.Results/Http/Writing/MetadataSerializationMode.cs Removed placeholder file (type moved to SharedJsonSerialization namespace)
Light.Results.Public.snk Added public key file for assembly signing
images/light-logo.png Added package icon
Light.Results.slnx Updated solution to include new workflow and README files
Various packages.lock.json files Updated with Microsoft.SourceLink.GitHub dependencies and version reference changes

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

---------------------------------

- Initial release 🚀
- LightActionResult and LightActionResult<T> and corresponding extension methods to turn result instances into HTTP success responses or RFC 9457 (and RFC 7807) compatbile Problem Details responses.
Copy link

Copilot AI Feb 24, 2026

Choose a reason for hiding this comment

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

Spelling error: "compatbile" should be "compatible".

Suggested change
- LightActionResult and LightActionResult<T> and corresponding extension methods to turn result instances into HTTP success responses or RFC 9457 (and RFC 7807) compatbile Problem Details responses.
- LightActionResult and LightActionResult<T> and corresponding extension methods to turn result instances into HTTP success responses or RFC 9457 (and RFC 7807) compatible Problem Details responses.

Copilot uses AI. Check for mistakes.
---------------------------------

- Initial release 🚀
- LightResult and LightResult<T> and corresponding extension methods to turn result instances into HTTP success responses or RFC 9457 (and RFC 7807) compatbile Problem Details responses.
Copy link

Copilot AI Feb 24, 2026

Choose a reason for hiding this comment

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

Spelling error: "compatbile" should be "compatible".

Suggested change
- LightResult and LightResult<T> and corresponding extension methods to turn result instances into HTTP success responses or RFC 9457 (and RFC 7807) compatbile Problem Details responses.
- LightResult and LightResult<T> and corresponding extension methods to turn result instances into HTTP success responses or RFC 9457 (and RFC 7807) compatible Problem Details responses.

Copilot uses AI. Check for mistakes.
Comment thread src/Directory.Build.props Outdated
<EmbedUntrackedSources>true</EmbedUntrackedSources>
<PackageTags>streaming;memory-management;form-file</PackageTags>
<PackageIcon>light-logo.png</PackageIcon>
<PackageReadmeFile>readme.md</PackageReadmeFile>
Copy link

Copilot AI Feb 24, 2026

Choose a reason for hiding this comment

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

The PackageReadmeFile references "readme.md" but the actual file is named "README.md" (uppercase). This case mismatch may cause issues on case-sensitive file systems. Update to "README.md" to match the actual filename.

Suggested change
<PackageReadmeFile>readme.md</PackageReadmeFile>
<PackageReadmeFile>README.md</PackageReadmeFile>

Copilot uses AI. Check for mistakes.
Comment thread .github/workflows/release-on-nuget.yml Outdated
- name: Push NuGet packages
env:
NUGET_API_KEY: ${{ secrets.NUGET_API_KEY }}
run: dotnet nuget push "./src/**/*.nupkg" --api-key $NUGET_API_KEY --source https://api.nuget.org/v3/index.json
Copy link

Copilot AI Feb 24, 2026

Choose a reason for hiding this comment

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

Consider adding the --skip-duplicate flag to the dotnet nuget push command. Without it, the workflow will fail if a package with the same version already exists on NuGet, which could happen during re-runs or if a release is republished.

Suggested change
run: dotnet nuget push "./src/**/*.nupkg" --api-key $NUGET_API_KEY --source https://api.nuget.org/v3/index.json
run: dotnet nuget push "./src/**/*.nupkg" --api-key $NUGET_API_KEY --source https://api.nuget.org/v3/index.json --skip-duplicate

Copilot uses AI. Check for mistakes.

- Initial release 🚀
- Result enrichment with ASP.NET Core's HttpContext
- Shared logic for Minimal API's and MVC's composition root
Copy link

Copilot AI Feb 24, 2026

Choose a reason for hiding this comment

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

Grammar issue: "Minimal API's" should be "Minimal APIs" (plural form, not possessive). Similarly, "MVC's" should be "MVC's" or simply "MVC".

Suggested change
- Shared logic for Minimal API's and MVC's composition root
- Shared logic for Minimal APIs and MVC's composition root

Copilot uses AI. Check for mistakes.
Comment thread src/Directory.Build.props Outdated
<IncludeSymbols>true</IncludeSymbols>
<SymbolPackageFormat>snupkg</SymbolPackageFormat>
<EmbedUntrackedSources>true</EmbedUntrackedSources>
<PackageTags>streaming;memory-management;form-file</PackageTags>
Copy link

Copilot AI Feb 24, 2026

Choose a reason for hiding this comment

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

The PackageTags value "streaming;memory-management;form-file" appears to be incorrect for a Result Pattern library. These tags suggest file upload/streaming functionality, which doesn't match the library's purpose. Consider using tags like "result-pattern;error-handling;functional-programming;aspnetcore;cloudevents" instead.

Suggested change
<PackageTags>streaming;memory-management;form-file</PackageTags>
<PackageTags>result-pattern;error-handling;functional-programming;aspnetcore;cloudevents</PackageTags>

Copilot uses AI. Check for mistakes.
@feO2x
Copy link
Copy Markdown
Owner Author

feO2x commented Feb 24, 2026

@copilot open a new pull request to apply changes based on the comments in this thread

Copy link
Copy Markdown
Contributor

Copilot AI commented Feb 24, 2026

@feO2x I've opened a new pull request, #21, to work on those changes. Once the pull request is ready, I'll request review from you.

Copilot AI and others added 3 commits February 24, 2026 11:32
Fix review feedback: spelling, package tags, readme casing, NuGet push flag
@github-actions
Copy link
Copy Markdown

Code Coverage

Package Line Rate Branch Rate Complexity Health
Light.Results 96% 94% 2300
Light.Results.AspNetCore.MinimalApis 93% 80% 25
Light.Results.AspNetCore.Mvc 93% 80% 26
Light.Results.AspNetCore.Shared 94% 92% 28
Summary 96% (3565 / 3703) 94% (1839 / 1962) 2379

Minimum allowed line rate is 60%

@feO2x feO2x merged commit d002f3e into main Feb 24, 2026
2 checks passed
@feO2x feO2x deleted the 19-release-preparations branch February 24, 2026 15:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

0.1.0 Release Preparations

3 participants