Skip to content
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

add licenseVisitor and noassertion type #24

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

dabutvin
Copy link
Member

adds licenseVisitor option to scan

  • could plug in something like spdx-correct
  • I would like to be able to at least correct casing of the identifiers within an expression

adds noassertion as a parsed type

  • today the parser throws when it hits an unrecognized expression
  • the thinking here is that the resulting expression could still have info
  • an MIT license would still satisfy an expression like MIT OR CustomLicense
  • an MIT license would not satisfy an expression like MIT AND CustomLicense

@kemitchell
Copy link
Member

See #16

@dabutvin
Copy link
Member Author

👍 I'll have a look and see if I can just get that one up to date instead and then slim this one down to licenseVisitor

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