Move to use only one central cdk directory - #1556
Conversation
3d50a01 to
eaa0735
Compare
0e0c810 to
199d14c
Compare
akash1810
left a comment
There was a problem hiding this comment.
Not for this PR necessarily, but I think we could make further changes to the cdk directory to make it easier to work with:
- Pinning each dependency to a specific version
- Either using
yarnORnpm. Currently a lockfile for each exists 😱. - Updating dependencies
577f414 to
b57dc4b
Compare
Created a card for this |
b57dc4b to
0337784
Compare
akash1810
left a comment
There was a problem hiding this comment.
Changes to tag values only is as basically a no-op. Great work!
aracho1
left a comment
There was a problem hiding this comment.
Thank you for doing this! ✨
0337784 to
2772d11
Compare
| }, | ||
| ], | ||
| "EvaluationPeriods": 48, | ||
| "EvaluationPeriods": 24, |
There was a problem hiding this comment.
This looks like a bad rebase as it's undone the changes from https://github.com/guardian/mobile-n10n/pull/1563/files? We could run a diff again to double check?
There was a problem hiding this comment.
@akash1810 could you have one more look before I merge this? Thank you!
There was a problem hiding this comment.
@akash1810 could you have one more look before I merge this? Thank you!
Let's run a cdk diff. If it only shows tag changes, we're good.
…o the main cdk directory
Co-authored-by: Akash <akash1810@users.noreply.github.com>
Co-authored-by: Akash <akash1810@users.noreply.github.com>
2772d11 to
a56df28
Compare
What does this change?
Moved contents of extra cdk directory in notificationworkerlambda into the main cdk directory.
Positive effects of this are
How to test
In the cdk directory run with with mobile credential to check the changes that were introduced:
Snapshots stay the same
(There is one difference, the gu:cdk:version in the senderworker.test.ts.snap snapshot is now TEST)
We checked the changes to cloudformation and they correspond only to a version bump.
Cloudformation changeset json
How can we measure success?
Everything works as before