Skip to content

feat: add support for License Expression field#3

Merged
szabgab merged 1 commit intoszabgab:mainfrom
eyalch:support-license-expression
Jan 9, 2026
Merged

feat: add support for License Expression field#3
szabgab merged 1 commit intoszabgab:mainfrom
eyalch:support-license-expression

Conversation

@eyalch
Copy link
Contributor

@eyalch eyalch commented Jan 9, 2026

Resolves #1

@szabgab szabgab merged commit a8402ca into szabgab:main Jan 9, 2026
2 checks passed
@szabgab szabgab requested a review from Copilot January 9, 2026 08:13
Copy link

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 PR adds support for the new license_expression field from PyPI package metadata, preferring it over the legacy license field when available.

  • Adds license_expression field to data structures
  • Implements fallback logic to use license_expression first, then fall back to license
  • Updates serialization/deserialization to handle the new field

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File Description
src/lib.rs Adds license_expression field to the MyProject struct
src/download.rs Adds license_expression field to the Info struct and maps it when creating MyProject instances
src/report.rs Implements preference logic to use license_expression over legacy license field in license reporting

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

@eyalch eyalch deleted the support-license-expression branch January 9, 2026 09: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.

collect license from "License Expression" field

3 participants