Skip to content

Update node and dependencies#146

Merged
codentacos merged 24 commits intomainfrom
DTBTWEB-1160-update-deps
Feb 26, 2026
Merged

Update node and dependencies#146
codentacos merged 24 commits intomainfrom
DTBTWEB-1160-update-deps

Conversation

@codentacos
Copy link
Member

@codentacos codentacos commented Feb 19, 2026

Summary of changes

  • Update to node v24
  • Updates dependencies
    • braintree/browser-detection to 2.1.1
    • @playwright/test to 1.58.2
    • @types/jest to 30.0.0
    • @typescript-eslint/eslint-plugin to 8.56.0
    • browserify to 17.0.1
    • browserstack-local to 1.5.10
    • dotenv to 17.3.1
    • eslint to 9.39.2
    • eslint-config-braintree to 7.0.1
    • eslint-plugin-prettier to 5.5.5
    • express to 5.2.1
    • jest to 30.2.0
    • jest-environment-jsdom to 30.2.0
    • jest-util to 30.2.0
    • jsdoc to 4.0.5
    • playwright to 1.58.2
    • ts-jest to 29.4.6
  • adds dev dependencies:
    • prettier 3.8.1

Checklist

  • Added a changelog entry
  • Relevant test coverage
  • Tested and confirmed flows affected by this change are functioning as expected

Authors

List GitHub usernames for everyone who contributed to this pull request.

Reviewers

@braintree/team-sdk-js

@github-actions
Copy link
Contributor

github-actions bot commented Feb 19, 2026

Dependency Review

The following issues were found:
  • ✅ 0 vulnerable package(s)
See the Details below.

Scanned Files

  • package-lock.json

@codentacos codentacos marked this pull request as ready for review February 25, 2026 15:41
@codentacos codentacos requested a review from a team as a code owner February 25, 2026 15:41
Copy link
Contributor

@adrmachado-public adrmachado-public left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

i don't particularly like having to pull eslint-disable-next-line no-unused-vars so much, but i also don't mind putting that off until we start converting to typescript to see what's actually necessary

@codentacos
Copy link
Member Author

@adrmachado-public I agree and am not a fan either. These were not being flagged as issues before package upgrades which is why there are new uses of these ignores. I suspect it's probably changes that happened in eslint-config-braintree because this was a major version upgrade. But we will definitely look to reduce the usages of these as we make changes in this repo.

Copy link
Contributor

@GoogilyBoogily GoogilyBoogily left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍 🐸

@codentacos codentacos merged commit 6fd1779 into main Feb 26, 2026
6 checks passed
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.

3 participants