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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore(deps): bump type deps, fix lock, add node engine #439

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

chore(deps): bump type deps, fix lock, add node engine #439

wants to merge 3 commits into from

Conversation

sad-squid
Copy link

@sad-squid sad-squid commented Dec 21, 2020

  • Added a node engine to package.json since there was no node version specified in README or package.json
  • Updated type dependencies to match deps versions of react and react-dom
  • Running yarn install cleaned up the audio-activity dep from the lock file
  • Upgrade @types/node to latest version

Hi all! I was looking to contribute since I wanted to help support this awesome project. Just fixing some quick things I saw right away in the project, feedback welcome!

EDIT:
I used Node v12 as the engine since there was a types/node package for v12. Looks like the build has failed and the remote image expects 14 so I will adjust both the type dependency and the engine config to v14.

Adjusting the engine to match the build pipeline with: ~14 seems to have worked 馃憤

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