> Can you please share some pointers ? Only `org.apache.pulsar.shell.ShellCommandsProvider#getJCommander` uses jcommander API. I need to use picocli instead of jcommander, or add a new method.
Using picocli instead of jcommander will reduce some code, but it will break the signature of some methods. I need to carefully study how to do it. This thread is just a preliminary idea to replace jcommander, and I will write a PIP later. Thanks, Zixuan Enrico Olivelli <eolive...@gmail.com> 于2024年2月23日周五 22:59写道： > Il Ven 23 Feb 2024, 04:34 Zixuan Liu <node...@gmail.com> ha scritto: > > > Thanks for the feedback! > > > > > Please take into account Pulsar Shell extensions > > > > This is a tricky issue, I noticed that it depends on jcommander here, > > > > Can you please share some pointers ? IIRC the shell extensions shouldn't > need jcommander, it should be exposed in the public API > > Enrico > > and I > > need to break this interface, and make it compatible with Pulsar Shell > > implementation. > > > > I will create a draft PR to migrate the CLI parser from jcommander to > > picocli, and then make a PIP to the Pulsar. > > > > Thanks, > > Zixuan > > >
