X Tutup
Skip to content

fix(Dart): make some playground samples run with Dart Dev Compiler#6441

Closed
jacob314 wants to merge 1 commit intoangular:masterfrom
jacob314:dart-dev-compiler-fixes
Closed

fix(Dart): make some playground samples run with Dart Dev Compiler#6441
jacob314 wants to merge 1 commit intoangular:masterfrom
jacob314:dart-dev-compiler-fixes

Conversation

@jacob314
Copy link
Copy Markdown
Contributor

Resolve all invalid field override errors, workaround current
reflection limitations in Dart Dev Compiler. todo, hello_world and
key_events samples now work with Dart Dev Compiler.

@yjbanov yjbanov self-assigned this Jan 12, 2016
@jacob314 jacob314 force-pushed the dart-dev-compiler-fixes branch 2 times, most recently from 6619186 to bae7cc0 Compare January 13, 2016 00:39
Resolve all invalid field override errors, workaround current
reflection limitations in Dart Dev Compiler. todo, hello_world and
key_events samples now work with Dart Dev Compiler.

BREAKING CHANGE: remove TemplateRef.elementRef setter
@jacob314 jacob314 force-pushed the dart-dev-compiler-fixes branch from a358ed7 to ed31812 Compare January 13, 2016 18:20
@yjbanov
Copy link
Copy Markdown
Contributor

yjbanov commented Jan 13, 2016

LGTM

@yjbanov yjbanov added pr_state: LGTM action: merge The PR is ready for merge by the caretaker labels Jan 13, 2016
@yjbanov
Copy link
Copy Markdown
Contributor

yjbanov commented Jan 13, 2016

A note on the breaking change for future reference. We've had the elementRef setter in TemplateRef by accident. We don't need it and nobody relies on its existence (see, for example, that TemplateRef_ does not implement a setter).

@mary-poppins
Copy link
Copy Markdown

Merging PR #6441 on behalf of @jeffbcross to branch presubmit-jeffbcross-pr-6441.

@mary-poppins
Copy link
Copy Markdown

Merging PR #6441 on behalf of @rkirov to branch presubmit-rkirov-pr-6441.

@mhevery mhevery closed this in 3e65d14 Jan 21, 2016
@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: 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

X Tutup