This differential move Android rescinds and badge updates to lib and makes the keyserver use shared code.
Diff Detail
Diff Detail
- Repository
- rCOMM Comm
- Lint
No Lint Coverage - Unit
No Test Coverage
Event Timeline
Comment Actions
I strongly encourage you to rethink the amount of type spreads you're doing, especially when they're nested within other types
| lib/push/notif-creators.js | ||
|---|---|---|
| 329 | In general, I am skeptical of this approach of nesting unions inside a spread. In the past I've seen Flow not handle it well. I usually opt to make the upper type a union, and to include the shared properties in each branch (senderDeviceID, platformDetails) via a spread | |
| 333 | ||
| lib/types/notif-types.js | ||
| 140 | What is this spread doing? | |
| 165 | What is this spread doing? | |