Skip to content

G3_MASTER: Fix types of _componentProviders in application_ref for dart#8680

Closed
tbosch wants to merge 6 commits into
angular:g3_masterfrom
tbosch:fix_types
Closed

G3_MASTER: Fix types of _componentProviders in application_ref for dart#8680
tbosch wants to merge 6 commits into
angular:g3_masterfrom
tbosch:fix_types

Conversation

@tbosch

@tbosch tbosch commented May 16, 2016

Copy link
Copy Markdown
Contributor

No description provided.

@mhevery mhevery added pr_state: LGTM action: merge The PR is ready for merge by the caretaker labels May 16, 2016
@tbosch tbosch force-pushed the fix_types branch 2 times, most recently from 534ffd3 to 9f6a962 Compare May 16, 2016 22:53
@mhevery mhevery self-assigned this May 16, 2016
@tbosch tbosch force-pushed the fix_types branch 3 times, most recently from ad4a3f9 to 3add20d Compare May 17, 2016 01:00
@tbosch tbosch changed the title Fix types of _componentProviders in application_ref for dart MERGE into G3_MASTER: Fix types of _componentProviders in application_ref for dart May 17, 2016
@googlebot

Copy link
Copy Markdown

We found a Contributor License Agreement for you (the sender of this pull request) and all commit authors, but as best as we can tell these commits were authored by someone else. If that's the case, please add them to this pull request and have them confirm that they're okay with these commits being contributed to Google. If we're mistaken and you did author these commits, just reply here to confirm.

@tbosch tbosch closed this May 17, 2016
@tbosch tbosch reopened this May 17, 2016
@tbosch tbosch changed the title MERGE into G3_MASTER: Fix types of _componentProviders in application_ref for dart G3_MASTER: Fix types of _componentProviders in application_ref for dart May 17, 2016
@googlebot

Copy link
Copy Markdown

CLAs look good, thanks!

tbosch added 3 commits May 17, 2016 11:13
The list is otherwise reified as `List<Provider>`
but non-Provider objects are later appended to it
tbosch added 2 commits May 17, 2016 11:13
This is not needed for syncing Dart into G3,
and we are testing against typescript_next
on the real master.
@tbosch tbosch force-pushed the fix_types branch 8 times, most recently from 1ca961e to 31ddd9c Compare May 17, 2016 22:08
tbosch added a commit that referenced this pull request May 17, 2016
@tbosch

tbosch commented May 18, 2016

Copy link
Copy Markdown
Contributor Author

Merged into g3_master as c896dd2 and following

@tbosch tbosch closed this May 18, 2016
@tbosch tbosch deleted the fix_types branch May 18, 2016 22:12
@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.

3 participants