Adam Wรณjcik
Adam Wรณjcik
ok I updated the spec @waldekmastykarz, @martinlingstuyl what do you think about it now?
> I'm actually using `helpMode` for #3782 ๐ How about `helpTarget`? done โ
@pnp/cli-for-microsoft-365-maintainers any other remarks or should we open this up ?
Ready to merge ๐ after next minor release @MathijsVerbeeck awesome work. I will do a small fixup when merging and resolve the conflicts as well
merged manually ๐
@martinlingstuyl looks really promising ๐ I think it will be a significant improvement in the codebase ๐ช. checked locally โ  @pnp/cli-for-microsoft-365-maintainers what do you think? I am all in...
> @Adam-it : Thanks for the awesome review, I learn a lot from the coding reviews. Hopefully I will finish reviewing all comments soon. No rush. Once again thank you...
@reshmee011 I added a few more comments (mostly tiny details) ๐. Please double-check that GitHub tends to collapse comments so it's easy to overlook them and I suspect that is...
@reshmee011 I rechecked your question with the `CommandError` and left the comment for it. Basically it was all ok and I failed to notice we are already hanling it ok....
Ready to merge ๐ I should resolve one small comment when merging