Skip to content

Conversation

mhegazy
Copy link
Contributor

@mhegazy mhegazy commented Jun 15, 2016

Take-2 Fixes #9173.

Clear out more settings, and add tests for all the settings that were not covered before.

options.noLib = true;

// Clear out the lib and types option as well
// Clear out other settings that would not be participate in transpiling this module
Copy link
Contributor

Choose a reason for hiding this comment

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

nit: would not be used

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done

@yuit
Copy link
Contributor

yuit commented Jun 17, 2016

@mhegazy you will want to merge master in sicne we have change our unittests/transpile to use baselines instead

@mhegazy
Copy link
Contributor Author

mhegazy commented Jun 17, 2016

@yuit can you take a look?

@yuit
Copy link
Contributor

yuit commented Jun 20, 2016

@mhegazy sorry didn't realize my comment on Friday didn't show up. The change looks good. though I think we should pull transpile option test out as the file is growing

@mhegazy
Copy link
Contributor Author

mhegazy commented Jun 20, 2016

i am not sure i see the need to split the file. they are all unit tests for the same API.

@mhegazy mhegazy merged commit 8b5616d into master Jun 20, 2016
@mhegazy mhegazy deleted the Fix9173-2 branch June 20, 2016 16:55
@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.

4 participants