Conversation
|
Do we need to update a cgmanifest JSON? |
I was trying to update several deps at once or I would of already done this one. I also verified this change did not cause any issues. But it brings more features and bug fixes that may make our current parser a bit better. I do seriously recommend allowing ingestion of TOML at some point, not just json as it could prove useful in extensions or settings in general. |
|
@DHowett None of the I'm not sure that we need any of the new features or if any of the bug fixes apply to us. I was mostly just doing a mechanical update this morning as I had my coffee. |
That one we should fix :) it’s not just for OS ingestion, but general project health. |
|
Okeydokey, that's easy enough |
|
@msftbot merge this in 1 minute |
|
Hello @DHowett! Because you've given me some instructions on how to help merge this pull request, I'll be modifying my merge approach. Here's how I understand your requirements for merging this pull request:
If this doesn't seem right to you, you can tell me to cancel these instructions and use the auto-merge policy that has been configured for this repository. Try telling me "forget everything I just told you". |
Updates CLI11 to 1.9. Doesn't look like any of the tests broke.