Skip to content

Comments

Combine config and kwargs#180

Merged
kylebarron merged 2 commits intomainfrom
kyle/combine-config-kwargs
Jan 30, 2025
Merged

Combine config and kwargs#180
kylebarron merged 2 commits intomainfrom
kyle/combine-config-kwargs

Conversation

@kylebarron
Copy link
Member

@kylebarron kylebarron commented Jan 30, 2025

This enables merging any parameters that are in config and kwargs for each store. If any config key is passed in both config and kwargs (even if spelled differently or cased differently), an error will be raised.

Closes #178

@kylebarron kylebarron enabled auto-merge (squash) January 30, 2025 19:11
@kylebarron kylebarron merged commit 5082d96 into main Jan 30, 2025
4 checks passed
@kylebarron kylebarron deleted the kyle/combine-config-kwargs branch January 30, 2025 19:13
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.

Validate that config parameters are passed either in the config dict or in kwargs but not both

1 participant