Skip to content

Conversation

@kbkk
Copy link

@kbkk kbkk commented Jun 10, 2021

Issue #, if available: N/A

Description of changes:

Support loading *.ts files

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

"author": "AWS Lambda",
"license": "Apache-2.0",
"scripts": {
"start": "node -r ts-node/register $(npm bin)/aws-lambda-ric index.handler"
Copy link
Author

@kbkk kbkk Jun 10, 2021

Choose a reason for hiding this comment

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

This is inconsistent with the echo project (using npx vs npm). Couldn't figure another way to load ts-node

Perhaps ts-node could be registered "automagically" with *.ts extensions? WDYT

@@ -0,0 +1,97 @@
version: 0.2
Copy link
Author

Choose a reason for hiding this comment

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

It seemed overkill to add it to the each of the build matrices, so I created a new spec that will test *.ts in just one configuration

@kbkk kbkk closed this Jan 31, 2022
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.

1 participant