Skip to content

cleanup test harness code#5655

Merged
vladima merged 2 commits into
masterfrom
harnessCleanup
Nov 13, 2015
Merged

cleanup test harness code#5655
vladima merged 2 commits into
masterfrom
harnessCleanup

Conversation

@vladima
Copy link
Copy Markdown
Contributor

@vladima vladima commented Nov 13, 2015

  1. use Path to store files in harness compiler host
  2. remove unused code, convert HarnessCompiler to module as it is does not use any state

@mhegazy
Copy link
Copy Markdown
Contributor

mhegazy commented Nov 13, 2015

thanks!

@mhegazy
Copy link
Copy Markdown
Contributor

mhegazy commented Nov 13, 2015

👍

Comment thread src/harness/harness.ts Outdated
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

inputFilesWiithPath

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

thanks

vladima added a commit that referenced this pull request Nov 13, 2015
@vladima vladima merged commit d2d2213 into master Nov 13, 2015
@vladima vladima deleted the harnessCleanup branch November 13, 2015 23:30
@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