You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I think the following is a perfectly valid use case, and I understand the argument against it: possible race conditions, async side effects, etc. But this is the nature of these callbacks, and, the rule is not enforced for .catch anyway, so we could argue the same thing for it too
In reality, this pattern is useful, and having it banned in the recommended config feels a little too intrusive for me
I can provide a PR
Steps to Reproduce
import("/assets/path/deps/utils.js").then((module)=>{window.movePicCarousel=module.movePicCarousel;// the last callback doesn't need to return anything}).catch(console.error);
Expected behavior: [What you expect to happen]
No errors (like when ignoreLastCallback: true)
Actual behavior: [What actually happens]
Error
The text was updated successfully, but these errors were encountered:
Description
I think the following is a perfectly valid use case, and I understand the argument against it: possible race conditions, async side effects, etc. But this is the nature of these callbacks, and, the rule is not enforced for
.catch
anyway, so we could argue the same thing for it tooIn reality, this pattern is useful, and having it banned in the recommended config feels a little too intrusive for me
I can provide a PR
Steps to Reproduce
Expected behavior: [What you expect to happen]
No errors (like when
ignoreLastCallback: true
)Actual behavior: [What actually happens]
Error
The text was updated successfully, but these errors were encountered: