Skip to content

Conversation

@Arlodotexe
Copy link
Member

This PR is a small change that closes #20.

@michael-hawker
Copy link
Member

@Arlodotexe I know your new test didn't repro it (confusingly), but did we want to include it anyway for other possible issues in the future?

@michael-hawker michael-hawker added dev loop ➰ For issues that impact the core dev-loop of building experiments sample app 🖼 source generator ⚙️ labels Mar 23, 2023
@Arlodotexe
Copy link
Member Author

Arlodotexe commented Mar 23, 2023

@Arlodotexe I know your new test didn't repro it (confusingly), but did we want to include it anyway for other possible issues in the future?

The test I had drafted didn't do anything new or different. It was a copy-paste of an existing test (but with the SettingsExpander sample/attributes) that I used for initial debugging. I couldn't repro the issue with it.

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

Labels

dev loop ➰ For issues that impact the core dev-loop of building experiments sample app 🖼 source generator ⚙️

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Multiple ToolkitBoolOptions causes crash

2 participants