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

Make sure you can't iterate a dictionary with forEach() #5467

Merged
merged 1 commit into from
Feb 22, 2023

Conversation

JacobOscarGunnarsson
Copy link
Contributor

@JacobOscarGunnarsson JacobOscarGunnarsson commented Feb 21, 2023

This test should validate that the following issue is resolved in bindgen

Copy link
Contributor

@elle-j elle-j left a comment

Choose a reason for hiding this comment

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

LGTM

@takameyer
Copy link
Contributor

@JacobOscarGunnarsson Just a quick note. In PR descriptions, please link to the github issue rather than the jira issue. For one, this has the benefit of closing said issue when merged, and for another, the user that submitted this github issue will see that there is progress for it.

Copy link
Contributor

@takameyer takameyer left a comment

Choose a reason for hiding this comment

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

LGTM!

@JacobOscarGunnarsson JacobOscarGunnarsson merged commit 046f43b into bindgen Feb 22, 2023
@JacobOscarGunnarsson JacobOscarGunnarsson deleted the jg/dictionary_foreach branch February 22, 2023 16:00
papafe added a commit that referenced this pull request Feb 24, 2023
* bindgen:
  Implement getAllSyncSessions (#5492)
  Ensure that Realm enums are accessible (#5484)
  Apply suggestions from code review [skip ci]
  Small corrections [skip ci]
  Added changelog and final corrections
  Stub
  add test to validate that foreach throws on a dictionary (#5467)
  Using `RealmInsertionModel` on `Results#update`
  Updated "react-native" dev dep to 0.71.0
  Bumped lower bound on our RN peer dependency
  [bindgen] SDK packaging (#5466)
  Adding "prebuild" and configuring it (#5447)
  add synthetic private brand fields to TS wrappers for C++ classes, and fix found bug
  import bindings directly rather than through internal
  Stub work
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Mar 15, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants