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: update devDependencies and handle resulting issues #510

Merged
merged 25 commits into from
Jun 5, 2024

Conversation

bnb
Copy link
Contributor

@bnb bnb commented Jun 3, 2024

Description

This PR updates almost all of the root devDependencies to their most recent version with the sole exception of ESLint because twilio-config-eslint hasn't updated to ESLint 9 yet. This PR also fixes all of the cascading issues, most of which were simply linter or formatting errors that came from changes introduced in the updated versions of everything. npm run test passes, npm run format passes.

Checklist

  • I ran npm test locally and it passed without errors.
  • I acknowledge that all my contributions will be made under the project's license.

Related issues

Any of the snyk/dependabot PRs that were about these packages. I think they'll self-close?

bnb and others added 25 commits June 3, 2024 11:46
Signed-off-by: Tierney Cyren <hello@bnb.im>
Signed-off-by: Tierney Cyren <hello@bnb.im>
Signed-off-by: Tierney Cyren <hello@bnb.im>
Signed-off-by: Tierney Cyren <hello@bnb.im>
…x later)

Signed-off-by: Tierney Cyren <hello@bnb.im>
…mplate-literals (fix later)

Signed-off-by: Tierney Cyren <hello@bnb.im>
Signed-off-by: Tierney Cyren <hello@bnb.im>
Signed-off-by: Tierney Cyren <hello@bnb.im>
Signed-off-by: Tierney Cyren <hello@bnb.im>
verify-prefill sample app
Signed-off-by: Tierney Cyren <hello@bnb.im>
Signed-off-by: Tierney Cyren <hello@bnb.im>
@cmsunu28 cmsunu28 merged commit 52a0064 into main Jun 5, 2024
2 of 3 checks passed
@cmsunu28 cmsunu28 deleted the bnb/update-root-devdeps-and-fix-test-failures branch June 5, 2024 22:20
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.

None yet

4 participants