New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: Introduce enforce-hash-update
flag to help hashing migration
#10209
Conversation
Codecov Report
@@ Coverage Diff @@
## master #10209 +/- ##
==========================================
+ Coverage 85.35% 85.38% +0.03%
==========================================
Files 339 339
Lines 13831 13836 +5
==========================================
+ Hits 11805 11814 +9
+ Misses 2026 2022 -4
Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Well done! Looks as a great migration aid
I've just spotted one styling issue in readme
docs/guides/upgrading-v3.md
Outdated
|
||
By default, Lambda version hashes will now be generated using improved algorithm (fixed determinism issues). As it is a breaking change that requires more manual effort during migration, it is still possible (but not recommended) to keep using old algorithm by using the following configuration: | ||
|
||
```` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think it should be one back quote less
docs/guides/upgrading-v3.md
Outdated
@@ -236,7 +260,7 @@ provider: | |||
statements: ... | |||
managedPolicies: ... | |||
deploymentRole: ... | |||
``` | |||
```` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
And then this change should not happen
5779071
to
5938c6a
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great 👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🎉 great!
I've used grammarly.com to add a few missing words (minor), and some suggestions to clarify a few parts.
5938c6a
to
0ed9b37
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
0ed9b37
to
25201f1
Compare
Reported internally, introduce a way to ease the lambda hashing algorithm migration