Skip to content

Update API samples and fix test issues#2599

Merged
mhegazy merged 5 commits into
masterfrom
APISamples
Apr 6, 2015
Merged

Update API samples and fix test issues#2599
mhegazy merged 5 commits into
masterfrom
APISamples

Conversation

@mhegazy
Copy link
Copy Markdown
Contributor

@mhegazy mhegazy commented Apr 2, 2015

Update the Samples to use new TS1.5 features like destructuring, let and const, and new es6 module syntax. also remove the content of typescript.d.ts from all baselines. should make test runs less noisy.

Conflicts:
	tests/baselines/reference/APISample_compile.js
	tests/baselines/reference/APISample_compile.types
	tests/baselines/reference/APISample_linter.js
	tests/baselines/reference/APISample_linter.types
	tests/baselines/reference/APISample_linter.types.pull
	tests/baselines/reference/APISample_transform.js
	tests/baselines/reference/APISample_transform.types
	tests/baselines/reference/APISample_watcher.js
	tests/baselines/reference/APISample_watcher.types
@DanielRosenwasser
Copy link
Copy Markdown
Member

👍

@DanielRosenwasser
Copy link
Copy Markdown
Member

Though please document the new harness flag.

Comment thread src/harness/harness.ts
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Keep space before {

@yuit
Copy link
Copy Markdown
Contributor

yuit commented Apr 4, 2015

👍

mhegazy added 2 commits April 6, 2015 13:38
Conflicts:
	tests/baselines/reference/APISample_compile.js
	tests/baselines/reference/APISample_compile.types
	tests/baselines/reference/APISample_linter.js
	tests/baselines/reference/APISample_linter.types
	tests/baselines/reference/APISample_transform.js
	tests/baselines/reference/APISample_transform.types
	tests/baselines/reference/APISample_watcher.js
	tests/baselines/reference/APISample_watcher.types
mhegazy added a commit that referenced this pull request Apr 6, 2015
Update API samples and fix test issues
@mhegazy mhegazy merged commit c53b0a5 into master Apr 6, 2015
@mhegazy mhegazy deleted the APISamples branch April 6, 2015 20:51
@microsoft microsoft locked and limited conversation to collaborators Jun 18, 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