Skip to content

feat: FunctionWithParamTokens.execute returns the value of the function#3132

Merged
kevmoo merged 1 commit into
angular:masterfrom
kevmoo:i3131_return_the_value
Jul 20, 2015
Merged

feat: FunctionWithParamTokens.execute returns the value of the function#3132
kevmoo merged 1 commit into
angular:masterfrom
kevmoo:i3131_return_the_value

Conversation

@kevmoo
Copy link
Copy Markdown
Contributor

@kevmoo kevmoo commented Jul 17, 2015

Closes #3131

Review on Reviewable

@kevmoo
Copy link
Copy Markdown
Contributor Author

kevmoo commented Jul 17, 2015

@tbosch @juliemr Are either of you allowed to approve such a pull request?

@juliemr
Copy link
Copy Markdown
Member

juliemr commented Jul 18, 2015

Looks good to me but vic or tobias should take a look as well.

@vsavkin vsavkin assigned kevmoo and unassigned vsavkin Jul 20, 2015
@vsavkin vsavkin added pr_state: LGTM action: merge The PR is ready for merge by the caretaker labels Jul 20, 2015
@kevmoo kevmoo merged commit 3dd05ef into angular:master Jul 20, 2015
@kevmoo kevmoo deleted the i3131_return_the_value branch July 20, 2015 16:03
@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 6, 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.

FunctionWithParamTokens.execute should return the result of apply

4 participants