Skip to content

ReactNative bug fix #244

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.

Sign up for GitHub

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

Jump to bottom
Merged
dkaminsky merged 1 commit into from
Mar 7, 2019
Merged

ReactNative bug fix #244

dkaminsky merged 1 commit into from
Mar 7, 2019

Conversation

@tkaye407
Copy link
Contributor

@tkaye407 tkaye407 commented Mar 7, 2019

No description provided.

dkaminsky
dkaminsky reviewed Mar 7, 2019
View reviewed changes
Copy link
Contributor

@dkaminsky dkaminsky left a comment •
edited
Loading

Choose a reason for hiding this comment

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

I’m not familiar with the semantic difference between the forEach vs the for loop here, can you explain it briefly?

Update: saw the answer you mentioned elsewhere - thanks

@coveralls
Copy link

coveralls commented Mar 7, 2019

Coverage Status

Coverage decreased (-0.3%) to 77.84% when pulling eb8aebe on tkaye407:reactNativeBugFix into 6ea7aeb on mongodb:master.

@dkaminsky dkaminsky merged commit d8aa84a into mongodb:master Mar 7, 2019
@tkaye407 tkaye407 deleted the reactNativeBugFix branch March 18, 2019 13:54
@tkaye407 tkaye407 restored the reactNativeBugFix branch March 18, 2019 13:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@dkaminsky dkaminsky dkaminsky left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@tkaye407 @coveralls @dkaminsky