Mark Li
Mark Li
I would like to work on this issue! Any suggestions on which modules to start first?
> Hi, thanks for your initial implementation. Here are some suggestions. > > This pr just implements consumer without enough tests. I recommend you implement producer first and verify it...
> I was talking about this with @JL03-Yue on Friday I think - the way we thought about approaching it was to model `dotnet tool update --all` as a completely...
@JL03-Yue Thank you for your help all the way through! I couldn't do this without your help and you have been so knowledgable! I have made the changes into subcommands...
> I will finish unit-test writing by the end of this weekend Done. One caveat is that user must use `dotnet update --all --global` instead of `dotnet update -g --all`...
> Would it allow `--global --all` if you set CliOption.Recursive = true for `--global` instead of adding `--global` to the subcommand separately? https://github.com/dotnet/command-line-api/blob/5ea97af07263ea3ef68a18557c8aa3f7e3200bda/src/System.CommandLine/CliOption.cs#L60-L63 Thank you for your input! You have...
> Would it allow `--global --all` if you set CliOption.Recursive = true for `--global` instead of adding `--global` to the subcommand separately? https://github.com/dotnet/command-line-api/blob/5ea97af07263ea3ef68a18557c8aa3f7e3200bda/src/System.CommandLine/CliOption.cs#L60-L63 Thank you for the suggestion again! This...
@JL03-Yue Thank you for helping me on the testing! I have updated the unit tests to include all senario with mocked file system
> > @JL03-Yue Thank you for helping me on the testing! I have updated the unit tests to include all scenario with mocked file system > > Thanks for the...
> Thanks for the update. I added some more details the [updated design details for update --all feature](https://github.com/dotnet/sdk/issues/37237) @JL03-Yue Thanks for updated change. I have added the unit testings for...