Skip to content

feat: enhance NuGet package metadata and extend .NET version support#3

Merged
j-d-ha merged 5 commits into
mainfrom
fix/update-workflows-again
Dec 19, 2025
Merged

feat: enhance NuGet package metadata and extend .NET version support#3
j-d-ha merged 5 commits into
mainfrom
fix/update-workflows-again

Conversation

@j-d-ha

@j-d-ha j-d-ha commented Dec 19, 2025

Copy link
Copy Markdown
Collaborator

🚀 Pull Request

📋 Summary

This PR improves the project's NuGet package configuration and extends .NET version compatibility:

  • Package Enhancements: Added README and icon to the NuGet package for better discoverability and branding
  • .NET Version Support: Extended build and PR workflows to test against .NET 8.0.x, 9.0.x, and 10.0.x (previously only 10.0.x)
  • Version Update: Bumped package version from alpha to beta release

✅ Checklist

  • Changes build cleanly
  • Tests added or updated (N/A - configuration changes only)
  • Documentation or README updated (README now included in package)
  • Coding style compliance
  • Local testing completion (where applicable)

🧪 Related Issues or PRs

💬 Notes for Reviewers

  • The multi-version .NET support ensures compatibility across .NET 8 LTS, .NET 9, and .NET 10
  • Icon and README files are now packaged but not visible in the solution explorer
  • Version change to beta indicates the package is approaching production readiness

…flows

- Updated .NET version matrix in `pr-build.yaml` and `build.yaml` to include 8.0.x and 9.0.x
- Maintains existing support for 10.0.x
- Changed `<VersionPrefix>` from `0.0.1-alpha` to `0.0.1-beta` in `Directory.Build.props`.
- Introduced `icon.png` as a new asset to the project.
- Added `<PackageReadmeFile>` property in `Directory.Build.props` to reference `README.md`.
- Configured `LayeredCraft.SourceGeneratorTools.csproj` to package `README.md` and `icon.png`.
- Updated `LayeredCraft.SourceGeneratorTools.UnitTests.csproj` to support .NET 8.0 and 9.0.
- Maintains existing compatibility with .NET 10.0.

@ncipollina ncipollina left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💰 LGTM!!

Comment thread Directory.Build.props
<Project>
<PropertyGroup>
<VersionPrefix>0.0.1-alpha</VersionPrefix>
<VersionPrefix>0.0.1-beta</VersionPrefix>

@ncipollina ncipollina Dec 19, 2025

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

❓ What do you think about making this 1.0.0-beta?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

I want to add it to one of our projects first to make sure it's working. I don't trust it yet lol.

@j-d-ha
j-d-ha merged commit 6e6aae0 into main Dec 19, 2025
1 check passed
@j-d-ha
j-d-ha deleted the fix/update-workflows-again branch December 19, 2025 18:38
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.

2 participants