Skip to content

Fix missing default/nil values related to object_of and array_of in bundle extensions#432

Merged
Blacksmoke16 merged 1 commit intomasterfrom
nt-defaults
Jul 29, 2024
Merged

Fix missing default/nil values related to object_of and array_of in bundle extensions#432
Blacksmoke16 merged 1 commit intomasterfrom
nt-defaults

Conversation

@Blacksmoke16
Copy link
Member

Context

Noticed this while working on #431. It seems the logic that was populating unprovided values, either default or nil, to the named tuple ended up in ValidateArguments. This made it so that the values appeared correct when printing the final CONFIG value, but was not done in time to be present at the time extensions are executed.

Changelog

  • Fix missing default/nil values related to object_of or array_of in bundle extensions

@Blacksmoke16 Blacksmoke16 added kind:bug An existing feature isn't working as expected component:dependency-injection labels Jul 29, 2024
@Blacksmoke16 Blacksmoke16 changed the title Populate NamedTuple configuration values with unprovided values Fix missing default/nil values related to object_of and array_of in bundle extensions Jul 29, 2024
@Blacksmoke16 Blacksmoke16 enabled auto-merge (squash) July 29, 2024 22:27
@Blacksmoke16 Blacksmoke16 merged commit d0c2bb4 into master Jul 29, 2024
@Blacksmoke16 Blacksmoke16 deleted the nt-defaults branch July 29, 2024 22:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

component:dependency-injection kind:bug An existing feature isn't working as expected

Development

Successfully merging this pull request may close these issues.

1 participant