Skip to content

Conversation

@hemangandhi
Copy link

From discussion in #139: we might as well just determine all the indeterminate values.

(Master-based version of #153, unlike #139 this does not include #146)

@hemangandhi hemangandhi requested a review from cielavenir May 9, 2024 01:10
@hemangandhi hemangandhi self-assigned this May 9, 2024
@cielavenir cielavenir requested a review from felixvd May 20, 2024 00:48
Copy link

@cielavenir cielavenir left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

so far it's working, thank you!

@cielavenir cielavenir requested a review from rdiankov May 30, 2024 09:06
@cielavenir
Copy link

@rdiankov please check this, without this the initial value is undefined

@rdiankov rdiankov merged commit 81155d8 into master Jun 10, 2024
@rdiankov rdiankov deleted the set-all-default-values-off-master branch June 10, 2024 09:47
@rdiankov
Copy link
Member

woot~ let's see if not changing version gets us in trouble

@felixvd
Copy link
Contributor

felixvd commented Jun 10, 2024

Not on here, we're only doing that on Gitlab 🙂

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants