-
-
Notifications
You must be signed in to change notification settings - Fork 43
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
Fixed issue with unknown extension when using allowArbitraryExtensions
compiler option
#313
Fixed issue with unknown extension when using allowArbitraryExtensions
compiler option
#313
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for your contribution! Please add a test case for this change, see https://github.com/timocov/dts-bundle-generator/tree/master/tests/e2e
https://github.com/timocov/dts-bundle-generator/tree/master/tests/e2e/test-cases/allow-js could be a good example of what you'd need to have. |
d63c8c0
to
4ec8de6
Compare
4ec8de6
to
f7d7a72
Compare
@stepankuzmin are you sure this fix fixes your problem? I just tried to run your test case against the version without your changes and it just works... Do you have a repro that I can take a look at? |
So we can validate the fix works
It seems changing extension of an entry file from |
allowArbitraryExtensions
compiler option
The fix has been published in 9.4.0 version. |
Skip files with unknown extensions. Otherwise,
dts-bundle-generator
will throw on.d.json.ts
files because it doesn't support overriding the type of a JSON file.