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

[4337] Feature: Upgrade dependencies #166

Merged
merged 1 commit into from
Nov 23, 2023
Merged

Conversation

mmv08
Copy link
Member

@mmv08 mmv08 commented Nov 22, 2023

This PR upgrades dependencies to the latest versions, even with a few major upgrades, no changes were required

@mmv08 mmv08 requested a review from a team as a code owner November 22, 2023 11:54
@mmv08 mmv08 requested review from rmeissner, nlordell, akshay-ap and remedcu and removed request for a team November 22, 2023 11:54
"@typescript-eslint/parser": "^5.28.0",
"@types/chai": "^4.3.11",
"@types/mocha": "^10.0.6",
"@types/node": "^20.9.4",
Copy link
Collaborator

Choose a reason for hiding this comment

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

We should update the CI to run on Node 20 instead of 18 then (so the types version matches what we actually use):

node-version: 18.x

@mmv08 mmv08 force-pushed the 4337/upgrade-dependencies branch from cc50715 to 574f42c Compare November 23, 2023 08:34
@mmv08 mmv08 requested a review from nlordell November 23, 2023 08:34
@coveralls
Copy link

Pull Request Test Coverage Report for Build 6967290847

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 100.0%

Totals Coverage Status
Change from base Build 6959417900: 0.0%
Covered Lines: 29
Relevant Lines: 29

💛 - Coveralls

@nlordell nlordell merged commit bdebd9f into master Nov 23, 2023
4 checks passed
@nlordell nlordell deleted the 4337/upgrade-dependencies branch November 23, 2023 09:12
@github-actions github-actions bot locked and limited conversation to collaborators Nov 23, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants