Skip to content

Remove "hoist-non-react-statics" from global type declaration #1480

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 4 commits into from
Aug 19, 2021

Conversation

saulve
Copy link
Contributor

@saulve saulve commented Aug 19, 2021

Description

Remove "hoist-non-react-statics" and "url-parse" from global type declaration.

Changelog

Changes typings/globalTypes.d.ts

@saulve saulve requested a review from ethanshar August 19, 2021 09:14
@@ -1,8 +1,6 @@
import React from 'react';
import {Appearance} from 'react-native';
//@ts-ignore
import hoistStatics from 'hoist-non-react-statics';
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

noticed that //@ts-ignore is used in a lot of places due to undeclared hoist-non-react-statics, so I removed those.

@@ -71,6 +71,7 @@
"@react-native-community/netinfo": "^5.6.2",
"@react-native-picker/picker": "^1.9.4",
"@testing-library/react-hooks": "^3.4.2",
"@types/hoist-non-react-statics": "^3.3.1",
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this is the main change (added to devDependencies)

@ethanshar ethanshar merged commit 077e495 into master Aug 19, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants