-
Notifications
You must be signed in to change notification settings - Fork 136
Enable (some?) cli bundled tools #428
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
Comments
I think the plan was to skip these for source-build anyway since they aren't yet source-buildable. Users of source-built will need to manually install them as global tools and the CLI is going to "include-list" these commands to inform users they need to install them. |
That sounds right to me. My concern is that there might be some bundled tools that are already source-buildable which are skipped because of the flag. I just noticed that these 5 are the only tools mentioned in |
I think we can close this. This is fixed in 3.1, right? |
@dseefeld @crummel @adaggarwal, can you confirm this was fixed in 3.1 (that the tools are now included) and close? |
Closing as all bundled tools that the MSFT build does - dev-certs, format, user-jwts, user-secrets, and watch. |
Bundled tool skip added in #418. I'm not sure how many bundled tools are skipped due to this:
source-build/repos/cli.proj
Lines 73 to 82 in a184e0b
The intention is to skip any that aren't built from source, but I don't know if there are some being skipped that don't need to be.
/cc @dleeapho @dseefeld @peterhuene
The text was updated successfully, but these errors were encountered: