Skip to content

Conversation

@nchen63
Copy link
Contributor

@nchen63 nchen63 commented Sep 26, 2016

Fixes #10933

@msftclas
Copy link

Hi @nchen63, 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!
We will now validate the agreement and then real humans will evaluate your PR.

TTYL, MSBOT;

@nchen63
Copy link
Contributor Author

nchen63 commented Sep 26, 2016

@mhegazy

@nchen63
Copy link
Contributor Author

nchen63 commented Sep 26, 2016

I didn't think this warranted a unit test, but I can add a simple one under test/cases/projects

@nchen63
Copy link
Contributor Author

nchen63 commented Sep 26, 2016

ok, added tests

@mhegazy
Copy link
Contributor

mhegazy commented Sep 26, 2016

you did not need a project test. a unit test with @declaration: true at the top would have done the job.

@mhegazy mhegazy merged commit 09eb6fe into microsoft:master Sep 26, 2016
@nchen63 nchen63 deleted the fix-export branch October 27, 2017 21:50
@microsoft microsoft locked and limited conversation to collaborators Jun 19, 2018
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.

3 participants