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

auspice-client: Speed up build #1041

Merged
merged 1 commit into from
Oct 21, 2024

Conversation

victorlin
Copy link
Member

@victorlin victorlin commented Oct 18, 2024

Description of proposed changes

Auspice's prepare script builds Auspice and by default will run during installation. It is not necessary here since there is a custom build call right after installation.

Notably, this avoids installation of devDependencies for git installs. One of the devDependencies, puppeteer, has recently been timing out on Heroku in attempts to download Chrome.

Related issue(s)

Checklist

  • Checks pass
  • Heroku build is successful (took ~3 minutes which seems to be an improvement)
  • Check if changes affect the resource index JSON revision
  • Applied change to #1039 to confirm that it resolves the issue with Heroku builds timing out (build completed in ~5 minutes):
    image

Auspice's prepare script builds Auspice and by default will run during
installation. It is not necessary here since there is a custom build
call right after installation.

Notably, this avoids installation of devDependencies for git installs.¹
One of the devDependencies, puppeteer, has recently been timing out on
Heroku in attempts to download Chrome.

¹ https://docs.npmjs.com/cli/v10/using-npm/scripts#life-cycle-scripts
@victorlin victorlin self-assigned this Oct 18, 2024
@nextstrain-bot nextstrain-bot temporarily deployed to nextstrain-s-victorlin--3uqvnj October 18, 2024 22:23 Inactive
@victorlin victorlin marked this pull request as ready for review October 18, 2024 22:34
Copy link
Member

@jameshadfield jameshadfield left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice debugging!

@victorlin victorlin changed the title Ignore scripts with npm ci before Auspice build auspice-client: Speed up build Oct 21, 2024
@victorlin victorlin merged commit fc6c41a into master Oct 21, 2024
7 checks passed
@victorlin victorlin deleted the victorlin/update-auspice-client-build branch October 21, 2024 15:19
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.

3 participants