Skip to content

modifications to angular flow#177

Merged
auchenberg merged 2 commits intomicrosoft:masterfrom
CannibalKush:CannibalKush-bg-patch
Feb 4, 2019
Merged

modifications to angular flow#177
auchenberg merged 2 commits intomicrosoft:masterfrom
CannibalKush:CannibalKush-bg-patch

Conversation

@CannibalKush
Copy link
Contributor

microsoft/vscode#64447

The tutorial as it stands creates a bit of ambiguity by defining a problem matcher for a background task, but not running the task in the background. Also, ng serve task was not defined as a preLaunch task. The guide instructed a user to follow a clunky flow.

Also, "identifiers" are deprecated and serve only to confuse new users. Therefore updated.

microsoft/vscode#64447

The tutorial as it stands creates a bit of ambiguity by defining a problem matcher for a background task, but not running the task in the background. Also, ng serve task was not defined as a preLaunch task. The guide instructed a user to follow a clunky flow.

Also, "identifiers" are deprecated and serve only to confuse new users. Therefore updated.
fix regex and ordering
@msftclas
Copy link

msftclas commented Dec 6, 2018

CLA assistant check
All CLA requirements met.

@CannibalKush
Copy link
Contributor Author

The guy that made volkswagen-lib is reviewing my PR, sick.

@CannibalKush
Copy link
Contributor Author

Fixes #159

@weinand
Copy link
Contributor

weinand commented Dec 6, 2018

Thanks for the great PR. Looks good to me.
@tonysneed Ok for you as well?

@auchenberg auchenberg merged commit 1b8c512 into microsoft:master Feb 4, 2019
@auchenberg auchenberg added this to the Dec/Jan 2019 milestone Feb 4, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants