Closed
Description
Describe the bug
In this recent PR:
this introduced a new clusterScailabilityType property which is spelled wrong and will cause confusion
Regression Issue
- Select this option if this issue appears to be a regression.
Last Known Working CDK Version
No response
Expected Behavior
I expected things to be spelled correctly.
Current Behavior
There is a typo, clusterScailabilityType should be clusterScalabilityType
Reproduction Steps
None, it's a typo
Possible Solution
replace all Scailability with Scalability
Additional Information/Context
No response
CDK CLI Version
Framework Version
No response
Node.js Version
OS
Language
TypeScript
Language Version
No response
Other information
No response
Activity
pahud commentedon Dec 9, 2024
Err...Yes we should get it fixed.
mazyu36 commentedon Dec 10, 2024
@badmintoncryer
FYI.
Sorry, I also missed this during the review.
badmintoncryer commentedon Dec 10, 2024
@mazyu36 Thanks! Of all things, I messed up with the argument name... I'm sorry @mbonig 😭
@pahud Is this a breaking change? Or should I add the correct name property and make the old property deprecated?
8 remaining items