Skip to content

Conversation

@ax-meyer
Copy link
Contributor

@ax-meyer ax-meyer commented Feb 27, 2025

What does this PR do?

Switches from using the license file to indicate the license in the nuget package to using PackageLicenseExpression to explicitly define the license for the nugets.

Why is it important?

This allows

How to test this PR

Manual test:

  • Run nuget pack for the project
  • Start manual upload process for a nuget here: https://www.nuget.org/packages/manage/upload
  • The page will show the package metadata with license information for verification before finishing the upload

@codecov
Copy link

codecov bot commented Feb 27, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 93.28%. Comparing base (422a54f) to head (332753b).
⚠️ Report is 1 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #1095   +/-   ##
=======================================
  Coverage   93.28%   93.28%           
=======================================
  Files         113      113           
  Lines        3439     3439           
  Branches      969      969           
=======================================
  Hits         3208     3208           
  Misses        215      215           
  Partials       16       16           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@atifaziz atifaziz linked an issue May 20, 2025 that may be closed by this pull request
@kchenery
Copy link

kchenery commented Oct 28, 2025

Hi,
Any reason this has not been merged yet?

cc @atifaziz

@atifaziz
Copy link
Member

atifaziz commented Nov 25, 2025

Hi, Any reason this has not been merged yet?

@kchenery Sorry, lack of time, but should managed to get it merged this week.

@atifaziz atifaziz merged commit d217ab1 into morelinq:master Nov 25, 2025
6 checks 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.

Switch to license expression

3 participants