WinterYukky

Results 30 comments of WinterYukky

Hi @pi0. I updated to automatically generate CDK code to the .output. Can you please re-review it?

I agree with @watany-dev's https://github.com/aws/aws-cdk/issues/20263#issuecomment-1213571723 . I know that I am currently blocking PR regarding the lambda layer. (because they are working on package size improvements for the lambda layer)....

I ran into the same problem and did debugged this issue. Perhaps this is due to the fact that integ-runner is v1. I found following code in the [`integ-runner`](https://github.com/aws/aws-cdk/blob/main/packages/%40aws-cdk/integ-runner/lib/runner/runner-base.ts#L358-L376). ```ts...

Thanks @jfuss for checking the PR. I was wondering why you use part of the cdk path (i.e Fn) for the ID value, but your explanation helped me understand. This...

@jfuss Of course! I'll try it.

> I left a couple comments but think I need to do a fresh pass when my brain is fresh. Just having a hard time following how `name` and `layer_id`...

Thanks for reviewing @moelasmar! Your propose that using new properties `custom_id` and `full_custom_id` looks like nice. I also understand your thoughts on other IaC frameworks. However, I don't agree to...

@mndeveci I'm sorry. I'm busy now but working this issue slowly. I'm never abandonment. When I'm ready let you know.

Thank you for your reviewing @danielroe! branch was out of date, so the latest changes were incorporated. Also, as mentioned in https://github.com/unjs/nitro/pull/240#discussion_r907183618, I have created a small utility package for...