Skip to content
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

Provide support for custom tsconfig files in serverless.ts definitions #8931

Closed
wants to merge 2 commits into from

Conversation

nhammond101
Copy link
Contributor

@nhammond101 nhammond101 commented Feb 11, 2021

Small change to allow the specifying of different tsconfig*.json files (tsconfig.app.json, tsconfig.serverless.json, etc) via the command line and the loading of tsconfig-paths if it's found within the project.

Primary use case is to support tsconfig paths, allowing the typescript compiler to use different paths when compiling the serverless.ts definition file.

Monorepos such as nx rely heavily on multiple modules referenced via the paths but currently serverless.ts won't be able to utilise it without tsconfig-paths

Addresses #8932

@codecov
Copy link

codecov bot commented Feb 11, 2021

Codecov Report

Merging #8931 (a68b503) into master (1eaa626) will decrease coverage by 0.11%.
The diff coverage is 50.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #8931      +/-   ##
==========================================
- Coverage   87.68%   87.56%   -0.12%     
==========================================
  Files         268      268              
  Lines       10041    10066      +25     
==========================================
+ Hits         8804     8814      +10     
- Misses       1237     1252      +15     
Impacted Files Coverage Δ
lib/configuration/read.js 75.26% <48.27%> (-12.97%) ⬇️
lib/Serverless.js 93.24% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 1eaa626...a68b503. Read the comment docs.

@medikoo
Copy link
Contributor

medikoo commented Feb 23, 2021

@nhammond101 great thanks for the effort. Still as discussed at serverless/typescript#28 ideally if we can keep any TS related handling externally and not in context of this repository

@medikoo medikoo closed this Feb 23, 2021
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