Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

fix: support configDir for other compilerOptions fields #84

Merged
merged 6 commits into from
Sep 12, 2024

Conversation

trikadin
Copy link
Contributor

This PR adds support for the "${configDir}" variable interpolation for all the fields in "compilerOptions" object as listed in the original issue

@trikadin
Copy link
Contributor Author

Sorry I didn't open an issue, it was faster to make a fix than a repro example :)

@privatenumber
Copy link
Owner

privatenumber commented Sep 11, 2024

No problem, thanks for the PR!

Can you add a test or update the existing one for configDir to include these properties?

@trikadin
Copy link
Contributor Author

trikadin commented Sep 11, 2024

No problem, thanks for the PR!

Can you add a test or update the existing one for configDir to include these properties?

Added test cases. Also, I fixed a bug when "declarationDir" was missing in generated "excludes", now that matches behaviour of TS :)

@privatenumber privatenumber changed the title Adds support for ${configDir} interpolation for all the compilerOptions fields fix: support configDir for other compilerOptions fields Sep 12, 2024
@privatenumber
Copy link
Owner

Ah nice catch! Thanks for the PR 🙌

@privatenumber privatenumber merged commit 7e58ea3 into privatenumber:master Sep 12, 2024
2 of 3 checks passed
@privatenumber
Copy link
Owner

🎉 This PR is included in version 4.8.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants