Skip to content

Update critical dependencies and simplify the build step#82

Merged
pcothenet merged 2 commits intomainfrom
pc/audit
Dec 6, 2022
Merged

Update critical dependencies and simplify the build step#82
pcothenet merged 2 commits intomainfrom
pc/audit

Conversation

@pcothenet
Copy link
Contributor

@pcothenet pcothenet commented Dec 6, 2022

What

  • Update a couple dependencies that had dependabot alert
  • Automatically run npm audit fix upon build
  • Simplify the build process (we only need a docker instance with node, multiple stages were a little superfluous)

Why

  • Fix vulnerabilities
  • Simplify the build

SDK Release Checklist

  • Have you added an integration test for the changes?
  • Have you built the package locally and made queries against it successfully?
  • Did you update the changelog?
  • Did you bump the package version?
  • If endpoints were removed, did you manually remove the corresponding files? (this should be rare)
  • For breaking changes, did you plan for the release of the new SDK versions and deploy the API to production?

@pcothenet pcothenet requested review from a team, holtbp, joustrich and kleinjm December 6, 2022 16:29
@pcothenet pcothenet merged commit 0159c44 into main Dec 6, 2022
@pcothenet pcothenet deleted the pc/audit branch December 6, 2022 17:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants