GH-3279: Cake runner arguments as build script arguments #4142
Conversation
augustoproiete
changed the title
|
Thanks @FrankRay78 tests look good. Let's wait for Spectre.Console vNext to ship and get this one merged 💪🏻 |
|
This PR is now blocked by #4157, "Upgrading to spectre.console 0.47.0 breaks the cake build" |
Hello @augustoproiete, I've now submitted the PR to upgrade spectre.console to the next version: #4236 . Apologies it's taken so long, there was a breaking change I found hard to work out. If / when the PR is merged, I'll look to rebase this branch and see about getting this PR over the line. Never did I expect #3279, which this PR relates to, to take more than a year...
So much upstream spectre.console CLI work was needed to be done first. |
…o spectre.console reference being 0.46.0
…or settings property, ConvertFlagsToRemainingArguments
…rmance problems when the integration tests are run after pushing to the PR
…time)" This reverts commit ba4e83f.
…he performance problems when the integration tests are run after pushing to the PR" This reverts commit 3b51ae7.
|
Hello @augustoproiete, I've now been able to complete this PR given spectre.console has been upgraded to 0.47.0. |
This PR fully addresses, #3279 "Allow users to use any of the Cake runner arguments as build script arguments" (and indirectly #2794, which should be closed)
The feature has been implemented for both Cake and also Frosting, unit tests written for each to exercise this new functionality and protect against any regressions.
eg.
cake --versionwill (continue to) run the built-in cake version feature,cake--version=1.2.3will run the build feature and make the argument and value accessible to the cake build script.This PR enables the passing of any of the built-in cake runner arguments, when found on the command line with an accompanying value, to the cake build script being executed.
** This PR will remain broken ** until spectre.console version 0.47.0 or above has been integrated, as it relies on a new configuration option that has been added to the spectre.console library particularly for this PR.
I will update the cake spectre.console package reference and push the change to this PR (fixing the build), once 0.47.0 is available on NuGet. FYI @patriksvensson