-
Notifications
You must be signed in to change notification settings - Fork 10
validate step title at plan time to avoid unexpected apply errors #250
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
Conversation
|
Code Coverage Artifact 📈: https://github.com/port-labs/terraform-provider-port-labs/actions/runs/14858836499/artifacts/3069537816Code Coverage Total Percentage:
|
🚨 The new code coverage percentage is lower than the current one. Current coverage: |
Code Coverage Artifact 📈: https://github.com/port-labs/terraform-provider-port-labs/actions/runs/14858823888/artifacts/3069545756Code Coverage Total Percentage:
|
🚨 The new code coverage percentage is lower than the current one. Current coverage: |
Code Coverage Artifact 📈: https://github.com/port-labs/terraform-provider-port-labs/actions/runs/14858942494/artifacts/3069738399Code Coverage Total Percentage:
|
🚨 The new code coverage percentage is lower than the current one. Current coverage: |
Code Coverage Artifact 📈: https://github.com/port-labs/terraform-provider-port-labs/actions/runs/14858942494/artifacts/3069772174Code Coverage Total Percentage:
|
🚨 The new code coverage percentage is lower than the current one. Current coverage: |
Code Coverage Artifact 📈: https://github.com/port-labs/terraform-provider-port-labs/actions/runs/14858942494/artifacts/3069816087Code Coverage Total Percentage:
|
🚨 The new code coverage percentage is lower than the current one. Current coverage: |
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.
Note: this PR is copy of #188 to run the test suite.
Description
What - Validate an action's step title to ensure it is at most 25 characters
Why - There's an undocumented character limit of 25 for the step title which causes a change to succeed to plan but then fail to apply.
How - Use
stringvalidator.LengthAtMost
validator for thetitle
attribute validationType of change
Please leave one option from the following and delete the rest: