Skip to content

Clarify the license due to bundled protobuf sources#92

Merged
andrewhickman merged 1 commit intoandrewhickman:mainfrom
LecrisUT:license
Apr 13, 2025
Merged

Clarify the license due to bundled protobuf sources#92
andrewhickman merged 1 commit intoandrewhickman:mainfrom
LecrisUT:license

Conversation

@LecrisUT
Copy link
Contributor

You already had the license file included, it's just that the metadata did not reflect it. Although you could make a symlink named LICENSE-protobuf and change the package.include so that it is clearer still

@codecov-commenter
Copy link

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 92.59%. Comparing base (40694f0) to head (9e0cbca).

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #92      +/-   ##
==========================================
- Coverage   92.63%   92.59%   -0.05%     
==========================================
  Files          20       20              
  Lines        2443     2443              
==========================================
- Hits         2263     2262       -1     
- Misses        180      181       +1     

☔ 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.

@andrewhickman
Copy link
Owner

Thanks. This makes me think these files should be distributed in a seperate, optional, crate in case users don't want to pick up the BSD license

@andrewhickman andrewhickman merged commit a76c61e into andrewhickman:main Apr 13, 2025
5 checks passed
@github-actions github-actions bot mentioned this pull request May 24, 2025
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