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

Update debugging.md #765

Merged
merged 2 commits into from
Aug 8, 2018
Merged

Update debugging.md #765

merged 2 commits into from
Aug 8, 2018

Conversation

lrkwz
Copy link
Contributor

@lrkwz lrkwz commented Aug 6, 2018

Updated docs

doc/debugging.md Outdated
@@ -4,7 +4,7 @@ Once you have set up a `launch.json` file with default configurations, you can m

![React Native launch configuration file](../images/launch-config.png)

For example, you can modify the `target` field to specify the simulator you want to target for iOS debugging.
For example, you can modify the `target` field to specify the simulator you want to target for iOS debugging or the device is as output from `adb devices`.
Copy link
Contributor

Choose a reason for hiding this comment

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

@lrkwz thanks for your suggestion, but I would change this to "or the device using the one from output of adb devices" to make it more clear to users. If you have any doubts/concerns please let me know.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes ot is more clear, sorry for my bad English.

Copy link
Contributor

Choose a reason for hiding this comment

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

@lrkwz no problem.

@ruslan-bikkinin
Copy link
Contributor

Hi @lrkwz and thanks for your contribution for React Native Tools! I've left a comment on your PR, so please take a look if you have time.

@codecov-io
Copy link

Codecov Report

Merging #765 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #765   +/-   ##
=======================================
  Coverage   14.39%   14.39%           
=======================================
  Files          63       63           
  Lines        3259     3259           
  Branches      416      416           
=======================================
  Hits          469      469           
  Misses       2790     2790

@ruslan-bikkinin ruslan-bikkinin merged commit fcfec28 into microsoft:master Aug 8, 2018
ruslan-bikkinin added a commit to ruslan-bikkinin/vscode-react-native that referenced this pull request Aug 28, 2018
ruslan-bikkinin added a commit that referenced this pull request Aug 28, 2018
* Revert "FIx incorrect runArguments assignment (#779)"

This reverts commit b9fbf69.

* Revert "Update changelog (#775)"

This reverts commit 5517641.

* Revert "Remove code push images (#771)"

This reverts commit b34bce9.

* Revert "Added support for Build Multiple Versions of React Native app (#766)"

This reverts commit 3e313f6.

* Revert "Update debugging.md (#765)"

This reverts commit fcfec28.

* Revert "fix typo (#763)"

This reverts commit 4dd439f.

* merge

* Revert "Use fork instead of spawn for AppWorker (#760)"

This reverts commit 2df05fa.

* Revert "Added some logging for debug (#759)"

This reverts commit 46e2461.

* Revert "[Android] Add reading SDK location from local.properties (#754)"

This reverts commit 43e1a99.

* Revert "Fixed error while stop expo packager (#753)"

This reverts commit 77b86bf.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants