Closed nicolasbender closed 4 months ago
Thanks for contributing to cloud_controller_ng. To speed up the process of reviewing your pull request please provide us with:
A short explanation of the proposed change: Fixes issues from https://github.com/cloudfoundry/cloud_controller_ng/pull/3831.
An explanation of the use cases your change solves The previous check did not have any effect: now there are really checks for nil-values and exclusion of nilable encrypted fields.
Links to any other associated PRs https://github.com/cloudfoundry/cloud_controller_ng/pull/3831
[x] I have reviewed the contributing guide
[x] I have viewed, signed, and submitted the Contributor License Agreement
[x] I have made this pull request to the main branch
main
[x] I have run all the unit tests using bundle exec rake
bundle exec rake
[ ] I have run CF Acceptance Tests
Thanks for contributing to cloud_controller_ng. To speed up the process of reviewing your pull request please provide us with:
A short explanation of the proposed change: Fixes issues from https://github.com/cloudfoundry/cloud_controller_ng/pull/3831.
An explanation of the use cases your change solves The previous check did not have any effect: now there are really checks for nil-values and exclusion of nilable encrypted fields.
Links to any other associated PRs https://github.com/cloudfoundry/cloud_controller_ng/pull/3831
[x] I have reviewed the contributing guide
[x] I have viewed, signed, and submitted the Contributor License Agreement
[x] I have made this pull request to the
main
branch[x] I have run all the unit tests using
bundle exec rake
[ ] I have run CF Acceptance Tests