Skip to content

Update dependencies#1155

Merged
kmcginnes merged 4 commits intoaws:mainfrom
kmcginnes:update-dependencies
Sep 8, 2025
Merged

Update dependencies#1155
kmcginnes merged 4 commits intoaws:mainfrom
kmcginnes:update-dependencies

Conversation

@kmcginnes
Copy link
Collaborator

@kmcginnes kmcginnes commented Sep 5, 2025

Description

  • Update all dependencies to latest versions
    • Holding off on Tailwind and Zod updates as they are slightly more involved
  • Fix new formatting issues from updated Prettier version
  • Enable concurrency in ESLint, which is new in the latest version
  • Fix type issues with Pino logger
  • When logging error object, use first parameter of log function

Validation

  • Smoke tested the UI
  • Verified Cytoscape update fixes edge rendering issue from previous newer versions
  • Ensured logger changes in server work by forcing error to occur

Related Issues

Check List

  • I confirm that my contribution is made under the terms of the Apache 2.0
    license.
  • I have run pnpm checks to ensure code compiles and meets standards.
  • I have run pnpm test to check if all tests are passing.
  • I have covered new added functionality with unit tests if necessary.
  • I have added an entry in the Changelog.md.

@kmcginnes kmcginnes force-pushed the update-dependencies branch 2 times, most recently from 1a63253 to 6e31d4b Compare September 5, 2025 22:02
@codecov
Copy link

codecov bot commented Sep 8, 2025

Codecov Report

❌ Patch coverage is 0% with 4 lines in your changes missing coverage. Please review.
✅ Project coverage is 36.48%. Comparing base (6070a74) to head (533ae3d).
⚠️ Report is 50 commits behind head on main.

Files with missing lines Patch % Lines
...ges/graph-explorer-proxy-server/src/node-server.ts 0.00% 3 Missing ⚠️
...ackages/graph-explorer-proxy-server/src/logging.ts 0.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1155      +/-   ##
==========================================
+ Coverage   33.94%   36.48%   +2.53%     
==========================================
  Files         524      531       +7     
  Lines       21535    23561    +2026     
  Branches     1563     1928     +365     
==========================================
+ Hits         7310     8596    +1286     
- Misses      14111    14850     +739     
- Partials      114      115       +1     
Flag Coverage Δ
unittests 36.48% <0.00%> (+2.53%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@kmcginnes kmcginnes marked this pull request as ready for review September 8, 2025 20:04
@kmcginnes kmcginnes merged commit 3858b9b into aws:main Sep 8, 2025
3 of 4 checks passed
@kmcginnes kmcginnes deleted the update-dependencies branch September 8, 2025 21:16
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.

Update dependencies

2 participants