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

Update Long type usage and improve UUID generation in topic client #430

Merged
merged 1 commit into from
Nov 4, 2024

Conversation

polRk
Copy link
Member

@polRk polRk commented Nov 4, 2024

This pull request includes several updates to dependencies and refactors some code to improve functionality and maintainability. The most important changes include updating dependencies in package.json, replacing the uuid package with the native crypto module for generating UUIDs, and making type adjustments in topic-writer.ts.

Dependency Updates:

  • package.json: Updated multiple dependencies to their latest versions, including @grpc/grpc-js, jsonwebtoken, long, luxon, and typescript. [1] [2]

Code Refactoring:

Pull request type

Please check one the type of change your PR introduces:

  • Bugfix
  • Feature
  • Code style update (formatting, renaming)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • Documentation content changes
  • Other (please describe):

What does it do?

Why is it needed?

How to test it?

Related issue(s)/PR(s)

Copy link

github-actions bot commented Nov 4, 2024

🌋 Here are results of SLO test for Native ydb-nodejs-sdk general:

Grafana Dashboard

SLO-nodejs

@polRk polRk force-pushed the 2024-11-04-update-deps branch from 1d76cff to 30bb650 Compare November 4, 2024 13:37
@polRk polRk merged commit 058b5a4 into main Nov 4, 2024
5 checks passed
@polRk polRk deleted the 2024-11-04-update-deps branch November 4, 2024 13:38
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.

1 participant