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

fix(build): sqlite3 build error #691

Merged
merged 1 commit into from
Jan 20, 2021

Conversation

chriscpritchard
Copy link
Contributor

added resolutions such that node-sqlite3 depends on the latest version of node-gyp

Description

Fixes build error related to sqlite3 on some platforms by using the latest version of node-gyp as a dependency resolution

Todos

  • Sucessfully builds yarn build
  • (Not Required)Translation Keys yarn i18n:extract

@sct
Copy link
Owner

sct commented Jan 20, 2021

Let's get this rebased and we can merge it.

updated package.json and yarn.lock to include a version of node-gyp that works with python3 instead
of python2 for node-sqlite3
@chriscpritchard
Copy link
Contributor Author

Let's get this rebased and we can merge it.

done!

@sct sct merged commit 3a1f6d5 into sct:develop Jan 20, 2021
@sct
Copy link
Owner

sct commented Jan 20, 2021

@all-contributors please add @chriscpritchard for code

@allcontributors
Copy link
Contributor

@sct

I've put up a pull request to add @chriscpritchard! 🎉

@github-actions
Copy link

🎉 This PR is included in version 1.18.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

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

2 participants