Skip to content

Conversation

m2mathew
Copy link
Contributor

@m2mathew m2mathew commented Jul 14, 2021

NOTE

I changed the "main" branch from master to main. People might need to update their forks.

The issue

We are getting a lot of "vulnerability" warnings from GitHub about outdated packages

The solution

  • Upgrade packages to latest minor version
    • Need to do more, but the transition to Gatsby v3 needs to be its own PR work
  • Add two new scripts for some handy helpers and update a third one:
    • dev: runs the app (used to be develop)
    • clean:gatsby: runs gatsby clean
    • clean:purge: removes all node modules then runs 'yarn cache clean`
  • Require node 12.x

Visuals

Before After
image image

Test cases

  1. Make sure you are on node version 12.x
  2. Get this code
  3. Run yarn
  4. Run yarn dev and make sure the app builds correctly

@m2mathew m2mathew added the enhancement New feature or request label Jul 14, 2021
@sriverag
Copy link

@m2mathew - Re: 0. Make sure you are on node version 12.x: We should add a .nvmrc in order to specify the supported Node version for the project.
It will be easier to enforce the version if using this shell script to do nvm use [version specified in .nvmrc] automatically: https://github.com/nvm-sh/nvm/blob/master/README.md#deeper-shell-integration

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants