-
Notifications
You must be signed in to change notification settings - Fork 101
Unable to compile TypeScript #30
Comments
Are you able to recreate the error in a container locally?
If you want to run
Asking because I'm using this action with a cdk TypeScript project without any issues. I'm taking a blind guess (being that I cannot see the |
Hi @cynicaljoy just ran your first example through container locally in a new cdk project that was initialized with the following command:
and still getting the same error as above with a default project.
|
I'm having the same issue in this run: https://github.com/gomint/aws/runs/1823323408?check_suite_focus=true. The package.json has been generated with When i manually install with the docker example you gave it works. At least |
I also came across using a project setup with I tried reproducing it in a local Docker container using the commands in #30 (comment) and succeeded. Curiously when I looked in to I tried to debug what was going on by running through the steps again, but the second time around Docker would just fail to connect (I blame WSL). From what I can tell, it seems like The same appears to be true for the For what it's worth, I am using That said, having fixed the above issues I now run into:
|
For what it's worth, I just ended up forking this and switching to using |
@cynicaljoy Any chances of solving this issue? |
Hi all, just to add some info about it. @birtles I can confirm that moving every dependencies to devDependencies did the trick for me (fixed my For the record, I'm using yarn (I knew that using yarn instead of npm would somewhere be a pain in the *ss, since it's not even mentioned in the AWS documentation...). Anyway, if this can help someone |
…n dev dependencies to dev dependencies in an attempt to fix issue when using CDK with yarn: https://github.com/youyo/aws-cdk-github-actions/issues/30\#issuecomment-809348175
I have all the dependancies installed & already built
tsc
our code but still getting the following error:@aws-cdk/core
is defined in package.json so not really sure why it can't find itThe text was updated successfully, but these errors were encountered: