chargebee-typescript icon indicating copy to clipboard operation
chargebee-typescript copied to clipboard
verified publisher icon chargebee

fix: move `@types/q` to dependencies.

Open b-steel opened this issue 2 years ago • 2 comments

Since @types/q is listed in devDependencies then it is not installed when chargebee-typescript is installed. Projects that use chargebee-typescript can have problems with the typescript compilation without this package and it can be very frustrating to debug.

The RequestWrapper components .request() method returns a Promise that is imported from the q package. Since chargebee-tyepscript is exporting this interface/type from the @types/q package it should be listed as a dependency, not a devDependency.

b-steel avatar Aug 29 '23 00:08 b-steel

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

sonarqubecloud[bot] avatar Oct 31 '23 00:10 sonarqubecloud[bot]

Looks like this has been added to 2.34.0 https://github.com/chargebee/chargebee-typescript/commit/8a4e9fa1a886565705650ac72120aa0842a73dc0#diff-7ae45ad102eab3b6d7e7896acd08c427a9b25b346470d7bc6507b6481575d519R23-L27 and it totally breaks typings https://github.com/chargebee/chargebee-typescript/issues/55

ihmpavel avatar Jul 03 '24 21:07 ihmpavel