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

npmignore: Don't include the source of tests #5799

Merged
merged 1 commit into from
Oct 8, 2021
Merged

Commits on Oct 8, 2021

  1. npmignore: Don't include the source of tests

    We know of no reason that including the source of tests in built npm
    packages would be helpful, and we've heard reports that including the
    RSA private key fixture as we do in apollo-server-core can trigger
    security scans.
    
    Change how we drop tests from "dist" to drop the whole test directory in
    case some other files sneak in there.
    
    Add a comment to the one npmignore file that differs from the others.
    
    Fixes #5781.
    glasser committed Oct 8, 2021
    Configuration menu
    Copy the full SHA
    fd3462c View commit details
    Browse the repository at this point in the history