Happy to make changes if there's something I'm missing in definition of done. Looked at https://aws.github.io/aws-pdk/contributing/index.html

Happy to make changes if there's something I'm missing in definition of done. Looked at https://aws.github.io/aws-pdk/contributing/index.html

Thanks so much for contributing @weklund ! You'll need to update the snapshots for the tests to pass - are you able to run pnpm build locally? That should run the tests and update the snapshots. If you run into trouble I can do this and push to your PR :)

Ooh also please can you mark your PR as a fix as chore will skip publishing to NPM https://github.com/aws/aws-pdk/blob/mainline/.github/workflows/release.yml#L47

Can do! @cogwirrel

Looks like the maven publish failed on mainline? Should I have done anything differently @cogwirrel ?