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

build: update docker images #237

Merged
merged 7 commits into from
May 21, 2022
Merged

build: update docker images #237

merged 7 commits into from
May 21, 2022

Conversation

erickzhao
Copy link
Member

@erickzhao erickzhao commented May 20, 2022

After making a trivial PR (#236), I realized that the CircleCI job for Linux was consistently failing.

I made a few upgrades and fixes:

  • Run sudo apt-get update before sudo apt-get install.
  • Migrate from deprecated circleci/node:XX images to the newer cimg/node:XX.YY.ZZ images. Needed to also install xvfb here because I think the newer images don't come with it installed.
  • Move tests from Node 10/12 to 14/16 (current maintenance/LTS versions).

@erickzhao erickzhao changed the title test build: fix circleci builds May 20, 2022
@erickzhao erickzhao changed the title build: fix circleci builds build: fix CI builds May 20, 2022
@erickzhao erickzhao marked this pull request as ready for review May 20, 2022 23:32
@erickzhao erickzhao changed the title build: fix CI builds build: update docker images May 20, 2022
@erickzhao erickzhao requested review from a team and malept May 20, 2022 23:35
@MarshallOfSound MarshallOfSound merged commit ef6d44b into main May 21, 2022
@MarshallOfSound MarshallOfSound deleted the erick/circleci-test branch May 21, 2022 23:23
@electron-bot
Copy link

🎉 This PR is included in version 3.2.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Spacetech pushed a commit to Spacetech/asar that referenced this pull request Aug 6, 2022
* build: upgrade docker images

* VERSION NUMBERS

* add apt-get update

* tweaks

* add xvfb

* update nodejs versions

* typo
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants