-
Notifications
You must be signed in to change notification settings - Fork 24
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
(Chore) Remove canvas to fix Netlify build #1572
Conversation
- name: Install required libraries | ||
run: sudo apt-get install build-essential libcairo2-dev libpango1.0-dev libjpeg-dev libgif-dev librsvg2-dev | ||
run: sudo apt-get install libgif-dev |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What is libgif-dev
needed for?
Closed because the PR caused unit tests to fail on CI - checking it out |
bd8d09c
to
9b374b1
Compare
@MauritsR FYI the author of |
👍 Was already looking at it - happy to see the binaries added so fast. Netlify now builds correctly (would have been quite a hassle to install the binaries there). Closing this PR but will open a new one to remove redundant binaries install. |
Removes canvas from package.json so that it's dependencies don't have to be installed (Dockerfile / ci)
This should also fix the Netlify build, which was not updated to perform the above installations.