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

[wmi] no warnings on 'Name' property 🔇 #2291

Merged
merged 1 commit into from
Feb 25, 2016
Merged

Conversation

yannmh
Copy link
Member

@yannmh yannmh commented Feb 25, 2016

Do not log a warning about non digit property values for the 'Name'
property.
Strengthen unit tests: less race condition error-prone.

@yannmh yannmh added the checks label Feb 25, 2016
@yannmh yannmh added this to the 5.7.0 milestone Feb 25, 2016
@olivielpeau
Copy link
Member

👍

Do not log a warning about non digit property values for the 'Name'
property.
Strengthen unit tests: less race condition error-prone.
@yannmh yannmh force-pushed the yann/wm-no-warning-on-names branch from 81e10c1 to d965bb5 Compare February 25, 2016 19:24
yannmh added a commit that referenced this pull request Feb 25, 2016
[wmi] no warnings on 'Name' property 🔇
@yannmh yannmh merged commit ba9d699 into master Feb 25, 2016
@yannmh yannmh deleted the yann/wm-no-warning-on-names branch February 25, 2016 20:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants