Skip to content

PR to review transformer changes from tbosch https://github.com/angular/angular/pull/5993#6171

Closed
kegluneq wants to merge 1 commit into
angular:masterfrom
kegluneq:ultraperf
Closed

PR to review transformer changes from tbosch https://github.com/angular/angular/pull/5993#6171
kegluneq wants to merge 1 commit into
angular:masterfrom
kegluneq:ultraperf

Conversation

@kegluneq
Copy link
Copy Markdown

/cc @tbosch , will assign to you when I have reviewed

@kegluneq
Copy link
Copy Markdown
Author

Please run dartfmt -w on all modified .dart files before committing

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: Since _extractReferencedType returns Iterable, consider using new Iterable.empty() here rather than a list literal to maintain consistent type

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.

Done

@kegluneq kegluneq added action: cleanup The PR is in need of cleanup, either due to needing a rebase or in response to comments from reviews and removed state: WIP labels Dec 29, 2015
@tbosch
Copy link
Copy Markdown
Contributor

tbosch commented Dec 29, 2015

Changes incorporated into #5993

@tbosch tbosch closed this Dec 29, 2015
@kegluneq kegluneq deleted the ultraperf branch December 30, 2015 21:28
@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

action: cleanup The PR is in need of cleanup, either due to needing a rebase or in response to comments from reviews cla: yes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants