Skip to content

fix: add license-expression case to get_license #124

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

mckornfield
Copy link

@mckornfield mckornfield commented Jul 9, 2025

Purpose of This Pull Request

Please check the relevant option by placing an "X" inside the brackets:

  • Documentation update
  • Bug fix
  • New feature
  • Other (please explain):

Overview of Changes

Provide a concise overview of the changes made in this pull request

Related Issue

Fixes #121

Reviewer Focus

Should be relatively straight forward, didn't see a great way to add a test without importing
another package but I can do so

@mckornfield mckornfield force-pushed the mck/fix-license-expression-case branch from 2ac0a6d to d18d16a Compare July 9, 2025 18:54
@mckornfield mckornfield changed the title fix: add license_expression case to get_license fix: add license-expression case to get_license Jul 9, 2025
@mckornfield
Copy link
Author

@FredHappyface I couldn't see a way to tag you for review other than a comment. Let me know if there's a more proper way I should ask for feedback (or how busy you are, etc.)

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.

Bug: License-Expression not found in local packages
1 participant