aws
fix(jsii-diff): renaming a positional argument is a breaking change in Python
Related: #2927
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
@RomainMuller do you think this has any chance of getting merged? I'm assuming the lack of progress is due to the high number of deprecations required on the CDK side.
@RomainMuller do you think this has any chance of getting merged? I'm assuming the lack of progress is due to the high number of deprecations required on the CDK side.
We are also consider alternate solutions to this...
Quite frankly... it might be "easier" to wait for Python 3.7 EOL next June, as Python 3.8+ supports "positional-only" argument notation (using the / modifier), which we could then adopt and forego this problem entirely.
@RomainMuller do you think this has any chance of getting merged? I'm assuming the lack of progress is due to the high number of deprecations required on the CDK side.
We are also consider alternate solutions to this...
Quite frankly... it might be "easier" to wait for Python 3.7 EOL next June, as Python 3.8+ supports "positional-only" argument notation (using the
/modifier), which we could then adopt and forego this problem entirely.
Got it. But that would still be a breaking change, some folks may be using them as keyword arguments.
Got it. But that would still be a breaking change, some folks may be using them as keyword arguments.
Yeah... Nothing is ever that easy... we might be able to strategize around this however.