vorpal icon indicating copy to clipboard operation
vorpal copied to clipboard

[2.0 Proposition] Rename vorpal.catch to avoid Promise ambiguity

Open AdrieanKhisbe opened this issue 6 years ago • 0 comments

I think we should take benefit of the inbuilding v2 to rename the .catch() commands. One of the reason is that is connote a promise method, error handling of other command. Besides it's usage is wrongly flagged as error by the common used promised eslint config with rule promise/catch-or-return

I'm not sure on which name we should use, but something around miss like onCommandMiss, or onNonMatchingCommand, or maybe better and simplerfallback

AdrieanKhisbe avatar Oct 07 '18 16:10 AdrieanKhisbe