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

Remove @types/long dependency #417

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ramikg
Copy link

@ramikg ramikg commented Nov 30, 2023

I've removed the @types/long dependency, in order to avoid the following deprecation warning:

This is a stub types definition. long provides its own type definitions, so you do not need this installed.

@Graphmaxer
Copy link

Graphmaxer commented Mar 4, 2024

Interested into this change since it break our builds

error TS2688: Cannot find type definition file for 'long'.
   The file is in the program because:
   Entry point for implicit type library 'long'

Currently, we fix it with overrides in the package.json

  "overrides": {
    "@types/long": "4.0.2"
  },

@Graphmaxer
Copy link

I created an issue in the Jira https://datastax-oss.atlassian.net/browse/NODEJS-671 Maybe it will help :/

@ramikg ramikg force-pushed the hotfix/remove-types-long-dependency branch from 6f21377 to 9217739 Compare October 26, 2024 12:27
@ramikg ramikg requested a review from nicoloboschi October 26, 2024 12:37
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