Skip to content
This repository was archived by the owner on May 20, 2025. It is now read-only.

alternative way to set property #141

Merged
merged 1 commit into from
Jan 9, 2016
Merged

Conversation

joenoon
Copy link
Contributor

@joenoon joenoon commented Jan 9, 2016

This is the only other way I can think of. re #139 and #140

This is the only other way I can think of.  re microsoft#139 and microsoft#140
@msftclas
Copy link

msftclas commented Jan 9, 2016

Hi @joenoon, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!

The agreement was validated by Microsoft and real humans are currently evaluating your PR.

TTYL, MSBOT;

@lostintangent
Copy link
Member

This is a great suggestion! I just verified that this does in fact work for 0.18 and earlier, so it's perfectly backwards compat :) I'm comfortable merging this in, but I'll wait for @geof90 to sign off as well.

@geof90
Copy link
Contributor

geof90 commented Jan 9, 2016

LGTM!

lostintangent added a commit that referenced this pull request Jan 9, 2016
alternative way to set property
@lostintangent lostintangent merged commit 75f2ed1 into microsoft:master Jan 9, 2016
This was referenced Jan 9, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants