Skip to content

docs(PipeTransform): added pure parameter - #4667

Closed
ocombe wants to merge 1 commit into
angular:masterfrom
ocombe:patch-2
Closed

ocombe wants to merge 1 commit into
angular:masterfrom
ocombe:patch-2

Conversation

@ocombe

@ocombe ocombe commented Oct 11, 2015

Copy link
Copy Markdown
Contributor

The documentation was outdated since alpha 38 when the pure parameter was added to the PipeMetadata.

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.

"... and set the pure parameter to fase"

@0x-r4bbit

Copy link
Copy Markdown
Contributor

Left a rewording proposal + backticks for code formatting!

Otherwise LGTM!

The documentation was outdated since alpha 38 when the `pure` parameter was added to the `PipeMetadata`.
@ocombe

ocombe commented Oct 11, 2015

Copy link
Copy Markdown
Contributor Author

Thanks, updated

@0x-r4bbit

Copy link
Copy Markdown
Contributor

👍

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.

Feel free to make one that I'll link here :)

@ocombe

ocombe commented Oct 24, 2015

Copy link
Copy Markdown
Contributor Author

The failed test has nothing to do with my update, could someone maybe relaunch it?

@vicb vicb added pr_state: LGTM action: merge The PR is ready for merge by the caretaker labels Oct 24, 2015
@mary-poppins

Copy link
Copy Markdown

Merging PR #4667 on behalf of @vicb to branch presubmit-vicb-pr-4667.

@mary-poppins mary-poppins removed the action: merge The PR is ready for merge by the caretaker label Oct 24, 2015
@vicb vicb self-assigned this Oct 27, 2015
vicb pushed a commit to vicb/angular that referenced this pull request Oct 27, 2015
closes angular#4667

The documentation was outdated since alpha 38 when the `pure` parameter
was added to the `PipeMetadata`.
@vicb vicb closed this in 7478105 Oct 28, 2015
@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 7, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants