Skip to content
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

Don't reconnect inspector if connection refused #22625

Closed
wants to merge 1 commit into from

Conversation

msand
Copy link
Contributor

@msand msand commented Dec 12, 2018

(Together with a pr to react-devtools) Fixes #21030

Changelog

[iOS] [Fixed] - Fix infinite retry loop of inspector

(Together with a pr to react-devtools) Fixes facebook#21030
@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Dec 12, 2018
@pull-bot
Copy link

Warnings
⚠️

📋 Test Plan - This PR appears to be missing a Test Plan.

⚠️

📋 Changelog - This PR appears to be missing Changelog.

Generated by 🚫 dangerJS

msand added a commit to msand/react-devtools that referenced this pull request Dec 12, 2018
@hramos hramos added the Missing Test Plan This PR appears to be missing a test plan. label Jan 11, 2019
@hramos
Copy link
Contributor

hramos commented Jan 11, 2019

This seems OK - let's wait for facebook/react-devtools#1248 to be merged.

@sunnylqm
Copy link
Contributor

@hramos Can we merge this first? I can not see why this pr should wait for the other?

@facebook-github-bot facebook-github-bot added the Import Started This pull request has been imported. This does not imply the PR has been approved. label Feb 21, 2019
Copy link
Contributor

@facebook-github-bot facebook-github-bot left a comment

Choose a reason for hiding this comment

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

@hramos is landing this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

@react-native-bot
Copy link
Collaborator

@msand merged commit d9489c4 into facebook:master.

@facebook facebook locked as resolved and limited conversation to collaborators Feb 21, 2019
@react-native-bot react-native-bot added the Merged This PR has been merged. label Feb 21, 2019
@hramos hramos removed Import Started This pull request has been imported. This does not imply the PR has been approved. labels Feb 23, 2019
sunnylqm pushed a commit to sunnylqm/react-native that referenced this pull request Feb 24, 2019
Summary:
(Together with a pr to react-devtools) Fixes facebook#21030

[iOS] [Fixed] - Fix infinite retry loop of inspector
Pull Request resolved: facebook#22625

Differential Revision: D14169392

Pulled By: hramos

fbshipit-source-id: 2e301fd9d458598b62399fc61a9859ad29928483
grabbou pushed a commit that referenced this pull request Feb 27, 2019
Summary:
(Together with a pr to react-devtools) Fixes #21030

[iOS] [Fixed] - Fix infinite retry loop of inspector
Pull Request resolved: #22625

Differential Revision: D14169392

Pulled By: hramos

fbshipit-source-id: 2e301fd9d458598b62399fc61a9859ad29928483
grabbou pushed a commit that referenced this pull request Feb 28, 2019
Summary:
(Together with a pr to react-devtools) Fixes #21030

[iOS] [Fixed] - Fix infinite retry loop of inspector
Pull Request resolved: #22625

Differential Revision: D14169392

Pulled By: hramos

fbshipit-source-id: 2e301fd9d458598b62399fc61a9859ad29928483
aleclarson pushed a commit to aleclarson/react-native-macos that referenced this pull request Jun 14, 2019
Summary:
(Together with a pr to react-devtools) Fixes facebook/react-native#21030

[iOS] [Fixed] - Fix infinite retry loop of inspector
Pull Request resolved: facebook/react-native#22625

Differential Revision: D14169392

Pulled By: hramos

fbshipit-source-id: 2e301fd9d458598b62399fc61a9859ad29928483
t-nanava pushed a commit to microsoft/react-native-macos that referenced this pull request Jun 17, 2019
Summary:
(Together with a pr to react-devtools) Fixes facebook#21030

[iOS] [Fixed] - Fix infinite retry loop of inspector
Pull Request resolved: facebook#22625

Differential Revision: D14169392

Pulled By: hramos

fbshipit-source-id: 2e301fd9d458598b62399fc61a9859ad29928483
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Bug CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. Merged This PR has been merged. Missing Test Plan This PR appears to be missing a test plan. Platform: iOS iOS applications.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Xcode 10 runtime warning on fresh install: nw_socket_handle_socket_event
6 participants