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

Speed up review app installation #1870

Merged
merged 3 commits into from
Oct 21, 2024

Commits on Oct 18, 2024

  1. Use Node.js version 20

    This change doesn't really affect the installation speed, but it's good
    to be consistent with the version used to build both nextstrain.org and
    auspice.us.
    victorlin committed Oct 18, 2024
    Configuration menu
    Copy the full SHA
    13a105f View commit details
    Browse the repository at this point in the history
  2. Don't run npm scripts

    This prevents running Auspice's prepare script, which is not necessary
    for installation without usage in the same environment.
    victorlin committed Oct 18, 2024
    Configuration menu
    Copy the full SHA
    c90a465 View commit details
    Browse the repository at this point in the history

Commits on Oct 21, 2024

  1. Set NODE_ENV=production during installation

    This prevents installation of devDependencies in the current project.
    There are none at the moment for both nextstrain.org and auspice.us, but
    if that ever changes in the future, we would want to ignore them in this
    step.
    victorlin committed Oct 21, 2024
    Configuration menu
    Copy the full SHA
    2aba0cf View commit details
    Browse the repository at this point in the history