Skip to content

Migrate getTodoComments to use the new tree#689

Merged
mhegazy merged 4 commits into
masterfrom
todoComments
Sep 18, 2014
Merged

Migrate getTodoComments to use the new tree#689
mhegazy merged 4 commits into
masterfrom
todoComments

Conversation

@mhegazy
Copy link
Copy Markdown
Contributor

@mhegazy mhegazy commented Sep 17, 2014

No description provided.

Comment thread src/services/services.ts Outdated
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.

how can this happen?

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.

you are right it can not. will remove it.

Comment thread src/services/services.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.

Use ts.map.

mhegazy added a commit that referenced this pull request Sep 18, 2014
Migrate getTodoComments to use the new tree
@mhegazy mhegazy merged commit 7aac8d5 into master Sep 18, 2014
@mhegazy mhegazy deleted the todoComments branch September 18, 2014 21:03
@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.

3 participants