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
fix: handle default string value properly #1243
Conversation
5e5b801
to
92ee8b9
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.
Couple of comments to look at. What do you think about adding some unit tests for this to catch any obvious mistakes?
@adschwartz is it cool if i do that in another PR because it would require me to add some plumbing - but this can unblock nam. |
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.
@Peeeekay Unblocking you but I think we should follow up with some increased quality and a unit test if possible.
🤖 I have created a release *beep* *boop* --- ## [0.82.20](0.82.19...0.82.20) (2023-09-06) ### Bug Fixes * handle default string value properly ([#1243](#1243)) ([6e49059](6e49059)) * Runtime values created by `add_services` were incorrect in the case of a skipped instruction ([#1239](#1239)) ([3412486](3412486)) --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please). --------- Co-authored-by: kurtosisbot <kurtosisbot@users.noreply.github.com>
This PR fixes the bug found recently where the default value in the starlark package was being overwritten by the "" passed by the frontend.