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

fix: dont omit license from stored manifests #7475

Merged
merged 1 commit into from
May 6, 2024
Merged

Commits on May 6, 2024

  1. fix: dont omit license from stored manifests

    This has the effect of adding licenses back into the lockfiles.
    Based on code in shrinkwrap.js and inventory.js, it appears that
    lockfiles are supposed to store the license. It's likely that in
    practice this behavior has not been consistent due to fetching
    of minifed manifests and packuments.
    
    I also attempted to remove the license code from shrinkwrap but
    that caused many more tests to break. Plus I believe this is the
    intended behavior, to have licenses in lockfiles based on bug
    reports like #7384
    lukekarrys committed May 6, 2024
    Configuration menu
    Copy the full SHA
    f6e0701 View commit details
    Browse the repository at this point in the history