Linus Unnebäck
Linus Unnebäck
I think this sounds very good, we should run a test and see how many packages it would break 👍
We should really write this down somewhere 😅 1. Clone https://github.com/standard/standard 2. edit `eslintrc.json` and add the rule 3. `npm it`
Could you post the entire list of failing repos, then we can start sending PRs 😄
Thank you! Cleaned it up a bit here: - [x] [fs-extra](https://github.com/jprichardson/node-fs-extra) - `fs-extra/lib/copy/copy.js:51:3`: Expected catch() or return (promise/catch-or-return) - [x] [karma-browserstack-launcher](https://github.com/karma-runner/karma-browserstack-launcher) - `karma-browserstack-launcher/index.js:203:5`: Expected catch() or return (promise/catch-or-return) - [x]...
> Regarding the `allowThen` option, can someone summarize the problem with using `.then(success, fail)` instead of `.then(success).catch(fail)`? If the functions `success` throws, the thrown error will be eaten and result...
Seems like this might be added directly to ESLint: https://github.com/eslint/eslint/issues/15240
I pinged in the eslint issue, let's see if there is an update...
Upstream has requested that we file a feature request here: https://github.com/nodesecurity/eslint-plugin-security @simone-sanfratello or @lmammino, would you be able to do this?
Hmmm, I'm not exactly sure what it does, would be nice with some examples... If it disallows this: ```js import { Alert, Platform, Text, View } from 'react-native' ``` in...
ping @alanshaw, any thoughts on merging this? ☺️