Skip to content
This repository has been archived by the owner on Sep 2, 2021. It is now read-only.

Merging master to release branch, will undo version bump in next PR #670

Closed
wants to merge 4 commits into from

Conversation

g-217
Copy link
Contributor

@g-217 g-217 commented Nov 18, 2019

With reference to issue adobe/brackets#14964, merging master to release branch, will undo version bumps in next PR.

shubhsnov and others added 4 commits May 2, 2019 13:47
…ingPort() needs a callback argument to work (#668)

* Disabled remote-debugging by default and appshell.app.getRemoteDebuggingPort() needs a callback argument to work

* Addressing review comments

* Addressing review comments

* Updating error meesage for port validation

* Addressing review comments from vickramdhawal, also it is not possible to use g_handler to hold remote-debugging-port as g_handler does not get initialized while we are parsing command line

* More error message correction
@g-217 g-217 changed the title Merging master to release branch, will undo version bumps in next PR Merging master to release branch, will undo version bump in next PR Nov 18, 2019
@sobisht
Copy link
Collaborator

sobisht commented Nov 18, 2019

@jha-G You should cherry pick the changes related to patch rather than merging master to release branch. In this way, version bump changes also would not be merged to the release branch.

@g-217
Copy link
Contributor Author

g-217 commented Nov 18, 2019

Closing the PR, as have raised #671.

@g-217 g-217 closed this Nov 18, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants