Skip to content

doc(NgSwitch): update API doc#4423

Closed
vicb wants to merge 1 commit into
angular:masterfrom
vicb:0929-doc
Closed

doc(NgSwitch): update API doc#4423
vicb wants to merge 1 commit into
angular:masterfrom
vicb:0929-doc

Conversation

@vicb
Copy link
Copy Markdown
Contributor

@vicb vicb commented Sep 30, 2015

No description provided.

@vicb vicb added comp: docs action: review The PR is still awaiting reviews from at least one requested reviewer and removed cla: yes labels Sep 30, 2015
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.

expression match --> expressions match / expression matches

@vicb
Copy link
Copy Markdown
Contributor Author

vicb commented Sep 30, 2015

@gkalpak thanks for the review

@gkalpak
Copy link
Copy Markdown
Member

gkalpak commented Sep 30, 2015

@vicb, thx for writing docs 😀

@vicb
Copy link
Copy Markdown
Contributor Author

vicb commented Sep 30, 2015

thx for writing docs

That's pure fun, you should try !

@IgorMinar
Copy link
Copy Markdown
Contributor

Vic please rebase.. sorry about the delay

@IgorMinar IgorMinar assigned vicb and unassigned IgorMinar Nov 13, 2015
@IgorMinar IgorMinar added state: WIP and removed action: review The PR is still awaiting reviews from at least one requested reviewer labels Nov 13, 2015
@vicb
Copy link
Copy Markdown
Contributor Author

vicb commented Nov 16, 2015

rebased

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.

what's the purpose of @private? we are not exporting the class, so it's private by default.

@IgorMinar
Copy link
Copy Markdown
Contributor

otherwise lgtm

@IgorMinar IgorMinar added the action: cleanup The PR is in need of cleanup, either due to needing a rebase or in response to comments from reviews label Nov 16, 2015
@vicb vicb added action: merge The PR is ready for merge by the caretaker and removed action: cleanup The PR is in need of cleanup, either due to needing a rebase or in response to comments from reviews state: WIP labels Nov 16, 2015
@mary-poppins
Copy link
Copy Markdown

Merging PR #4423 on behalf of @vsavkin to branch presubmit-vsavkin-pr-4423.

@vicb vicb closed this in 857bef9 Nov 17, 2015
@vicb vicb deleted the 0929-doc branch June 23, 2016 17:45
@angular-automatic-lock-bot
Copy link
Copy Markdown

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-bot angular-automatic-lock-bot Bot locked and limited conversation to collaborators Sep 8, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

action: merge The PR is ready for merge by the caretaker cla: yes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants