Skip to content

Conversation

@alexng353
Copy link

  • Added a new function to read and parse tsconfig.json for compiler
    options.
  • Updated the validateCompilerOptions function to merge options from
    tsconfig.json and tsoa.json, with precedence given to tsoa.json.
  • error handling for missing tsconfig.json files.
  • unit tests to validate the new functionality

- Added a new function to read and parse tsconfig.json for compiler
  options.
- Updated the validateCompilerOptions function to merge options from
  tsconfig.json and tsoa.json, with precedence given to tsoa.json.
- error handling for missing tsconfig.json files.
- unit tests to validate the new functionality
@alexng353
Copy link
Author

addresses #1825 and makes devx better

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hello there alexng353 👋

Thank you and congrats 🎉 for opening your first PR on this project.✨

We will review the following PR soon! 👀

@WoH
Copy link
Collaborator

WoH commented Jan 4, 2026

Thanks for this PR, I had this in the back of my head. Would you mind checking if we can use:

https://github.com/TypeStrong/tsconfig

or:

https://github.com/privatenumber/get-tsconfig?tab=readme-ov-file#how-can-i-use-typescript-to-parse-tsconfigjson

for the fallback?

Would be more consistent.

@alexng353
Copy link
Author

alexng353 commented Jan 4, 2026

I am leaning towards the second option since we wouldn't have to add another dependency. What do you mean by "fallback," though?

@alexng353
Copy link
Author

*Left unsquashed for more easy reading of the diff

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.

2 participants