6 Commits

Author	SHA1	Message	Date
Kevin Gozali	dd06f85bd0	TM JS: cleaned up TurboModuleRegistry types [2] ... Summary: Changelog: [Internal] Moved the imports for `TurboModuleRegistry` and `TurboModule` from `react-native`. This was a jscodeshift with the script: P120688078 Reviewed By: yungsters Differential Revision: D18262538 fbshipit-source-id: 48fac15229c897408928511c5ecbb42f17ec7b42	2019-11-04 18:51:05 -08:00
Emily Janzer	c10c147bcc	Fix a parameter type in NativeNetworkingAndroidSpec ... Summary: `responseType` should be a string, not an Object (which gets converted to a NativeMap by TM). Changelog: [General] [Fixed] Fix the flow type for NativeNetworkingModule Reviewed By: fkgozali Differential Revision: D18019418 fbshipit-source-id: 316470ca82241223eafb5b05a54fc2bbf3074821	2019-10-22 19:34:40 -07:00
Logan Daniels	91f139b941	xplat/js/react-native-github ... Reviewed By: panagosg7 Differential Revision: D16657770 fbshipit-source-id: 4e260842c838a35317515044c54ccf55a083da33	2019-08-09 10:11:15 -07:00
Michał Osadnik	28b282e5c7	Change callback in clearCookies and sendRequest from RCTNetworking to returning void ... Summary: Currently callback has type `callback: (result: boolean) => mixed` what is pointless (and breaks codegen), because value returning flow callback doesn't have any impact. Reviewed By: RSNara Differential Revision: D16282852 fbshipit-source-id: fe1036f17bff307ac91b280727eaa5bf81febd35	2019-07-17 06:21:49 -07:00
James Ide	33ee6f8b99	Add a lint rule to disallow Haste imports (#25058) ... Summary: This is an ESLint plugin that infers whether an import looks like a Haste module name. To keep the linter fast and simple, it does not look in the Haste map. Instead, it looks for uppercase characters in single-name import paths, since npm has disallowed uppercase letters in package names for a long time. There are some false negatives (e.g. "merge" is a Haste module and this linter rule would not pick it up) but those are about 1.1% of the module names in the RN repo, and unit tests and integration tests will fail anyway once Haste is turned off. You can disable the lint rule on varying granular levels with ESLint's normal disabling/enabling mechanisms. Also rewrote more Haste imports so that the linter passes (i.e. fixed lint errors as part of this PR). ## Changelog [General] [Changed] - Add a lint rule to disallow Haste imports Pull Request resolved: https://github.com/facebook/react-native/pull/25058 Differential Revision: D15515826 Pulled By: cpojer fbshipit-source-id: d58a3c30dfe0887f8a530e3393af4af5a1ec1cac	2019-05-30 07:45:16 -07:00
Michał Pierzchała	e8037cb942	Add spec for Networking (#24892) ... Summary: Part of #24875, adds a spec for Networking. Since `sendRequest` methods are different for both platforms, I had to create 2 spec files as Flow would merge their definitions even when I added `Platform.OS` check ## Changelog [General] [Added] - TM spec for Networking Pull Request resolved: https://github.com/facebook/react-native/pull/24892 Reviewed By: RSNara Differential Revision: D15543067 Pulled By: fkgozali fbshipit-source-id: 2b91114dfa45e7899bbb139656a30a6fd52e31db	2019-05-29 18:33:28 -07:00