| #
98ecfc87 |
| 24-Mar-2022 |
James Ide <[email protected]> |
[lint] Add "caughtErrors: 'all'" to linter config (#16591)
Why
---
ES2019 added "optional catch bindings" so that `e`/`error`/etc. don't need to be specified in catch clauses if the thrown value i
[lint] Add "caughtErrors: 'all'" to linter config (#16591)
Why
---
ES2019 added "optional catch bindings" so that `e`/`error`/etc. don't need to be specified in catch clauses if the thrown value isn't used. This changes our lint rules to warn if unnecessary error variables are declared. https://2ality.com/2017/08/optional-catch-binding.html.
In JS, as opposed to TS, we allow unused catch bindings if the variable names are prefixed with an underscore. This is so that JS targeting older JS interpreters can write `catch (_) {}` if needed. With TS we disallow unused catch bindings since tsc can compile code to its target JS version.
How
---
Added `caughtErrors: 'all'` to the no-unused-vars rule's configuration. As part of this change, I made the TypeScript linting rules apply only to .ts/.tsx/.ts.d files and updated the unit test snapshots.
Test Plan
---
Added unit tests to the linter. Ran linter over all modules in this repo with expotools.
show more ...
|
| #
35f78160 |
| 01-Sep-2021 |
Bartosz Kaszubowski <[email protected]> |
[eslint-config] remove `arrowParens` overwrite, reformat files (#14232)
|
| #
6da15324 |
| 27-Aug-2021 |
Bartosz Kaszubowski <[email protected]> |
[eslint-config] bump dependencies and minimum required lint tools versions (#13858)
Co-authored-by: James Ide <[email protected]>
|
| #
8285c032 |
| 13-Aug-2021 |
Tomasz Sapeta <[email protected]> |
Rename unimodules imports to expo-modules-core
|
| #
f8111c10 |
| 29-Jun-2021 |
Charlie Cruzan <[email protected]> |
[notifications] force requests to /updateDeviceToken and /getExpoPushToken to use lowercase deviceId (#13409)
* force requests to use lowercase device ID
* fix tests
* changelog
* Update pa
[notifications] force requests to /updateDeviceToken and /getExpoPushToken to use lowercase deviceId (#13409)
* force requests to use lowercase device ID
* fix tests
* changelog
* Update packages/expo-notifications/CHANGELOG.md
Co-authored-by: Expo CI <[email protected]>
Co-authored-by: Expo CI <[email protected]>
show more ...
|
| #
98629a0c |
| 23-Nov-2020 |
Stanisław Chmiela <[email protected]> |
[expo-notifications] Add autoreregistration device push token to Expo servers (#10908)
Please consult the pull request for full description and history of this commit.
|