-
Notifications
You must be signed in to change notification settings - Fork 89
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
Circular dependencies with TypeScript #348
Comments
yea, having the same issue. Did you have a chance to figure out how to fix it @jwrosewell? |
Thanks for confirming. Not yet looked at solving it. I'm concerned it's interfering with treeshaking and resulting in larger than required modules. |
Issues looks like it's related to this line.
I don't have the time to get a build environment working to make a PR. Others? |
Hi @jwrosewell try the latest version 1.5.3 and tell me if there is still this problem. |
In my case it is still there: Imports:
Build output:
|
Yup, we see this too!
|
-> @lamrowena |
1.4.0
Core
When transpiling in TypeScript the following warning is generated.
(!) Circular dependencies
..\node_modules@iabtcf\core\lib\encoder\index.js -> ..\node_modules@iabtcf\core\lib\encoder\SegmentEncoder.js -> ..\node_modules@iabtcf\core\lib\encoder\field\index.js -> ..\node_modules@iabtcf\core\lib\encoder\field\FieldEncoderMap.js -> ..\node_modules@iabtcf\core\lib\encoder\field\VendorVectorEncoder.js -> ..\node_modules@iabtcf\core\lib\encoder\index.js
..\node_modules@iabtcf\core\lib\encoder\index.js -> ..\node_modules@iabtcf\core\lib\encoder\SegmentEncoder.js -> ..\node_modules@iabtcf\core\lib\encoder\field\index.js -> ..\node_modules@iabtcf\core\lib\encoder\field\FieldEncoderMap.js -> ..\node_modules@iabtcf\core\lib\encoder\field\VendorVectorEncoder.js -> D:\Workspace\paf-mvp-implementation\node_modules@iabtcf\core\lib\encoder\index.js?commonjs-proxy -> ..\node_modules@iabtcf\core\lib\encoder\index.js
The module is included with the following import.
Do others have the same issue?
The text was updated successfully, but these errors were encountered: