-
Notifications
You must be signed in to change notification settings - Fork 585
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fake --help should document -t and -s parameters #2282
Comments
This is currently by design. The "targets" command line is entirely provided by a NuGet package and can be replaced. To "see" the targets command line use
In fact the regular
We could change the hint to |
I understand the technical details behind the current behavior. But it was non-discoverable to us today as beginners. I'd say a module should be able to extend the default |
Well technically we could make the hint more visible or something like that. The problem with other approaches is point in time. when you execute fake --help should we download and load packages just to give you help? I don't think users would accept that. |
Hm, no. But from a novice's point of view: We had the fake build script and a resolution already in place. Fake could attempt to load it and if it appears to be cached extend the help output by what the target module provides. |
While I agree in theory I just don't see how it could work technically, sorry |
The only thing that comes to mind is to 'special case' the targets module in some way and modify the output accordingly |
Another option would be to assume that targets are being used and hard-code its help into the fake executable. |
Yes indeed. |
If you want to optimize FAKE for being discoverable by beginners, then yes. |
What about this:
|
👍 |
Description
While
fake build
supports-t <Target>
and-s
to run a target without dependencies, both arguments are not reported when you runfake --help
or justfake
.Repro steps
Run
fake
(I'm using the global tool).Known workarounds
Read the docs.
Related information
The text was updated successfully, but these errors were encountered: