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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Don't use 'in' operator with Map #4916

Merged
merged 1 commit into from May 29, 2019
Merged

Don't use 'in' operator with Map #4916

merged 1 commit into from May 29, 2019

Conversation

fredj
Copy link
Member

@fredj fredj commented May 29, 2019

const map = new Map();
console.log('item' in map);
map.set('item', 'yes !')
console.log('item' in map);

prints:

false
false

馃槩

@fredj fredj merged commit 2cb3e0c into 2.4 May 29, 2019
@fredj fredj deleted the api_fixes branch May 29, 2019 15:16
@sbrunner sbrunner added this to the 2.4.1 milestone Jul 1, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants