X Tutup
Skip to content

fix(dart/reflection): Fix NoReflectionCapabilities interface#5578

Closed
kegluneq wants to merge 1 commit intoangular:masterfrom
kegluneq:dart_fix_reflection
Closed

fix(dart/reflection): Fix NoReflectionCapabilities interface#5578
kegluneq wants to merge 1 commit intoangular:masterfrom
kegluneq:dart_fix_reflection

Conversation

@kegluneq
Copy link
Copy Markdown

@kegluneq kegluneq commented Dec 3, 2015

Make NoReflectionCapabilities conform to the PlatformReflectionCapbilities
api, which prevents some confusing error messages.

Closes #5559

@yjbanov yjbanov added pr_state: LGTM action: merge The PR is ready for merge by the caretaker labels Dec 3, 2015
@jelbourn
Copy link
Copy Markdown
Contributor

jelbourn commented Dec 3, 2015

@kegluneq is the failure on Travis legitimate?

@kegluneq
Copy link
Copy Markdown
Author

kegluneq commented Dec 3, 2015

Yes and no 😃
This technically is a change to the public api, but in this case I believe the test is just out of date. I'll update it and check with @tbosch

@kegluneq kegluneq removed the action: merge The PR is ready for merge by the caretaker label Dec 3, 2015
Make `NoReflectionCapabilities` conform to the `PlatformReflectionCapbilities`
api, which prevents some confusing error messages.

Closes angular#5559
@kegluneq kegluneq force-pushed the dart_fix_reflection branch from 7c23188 to 238357a Compare December 3, 2015 19:33
@kegluneq kegluneq assigned tbosch and unassigned yjbanov Dec 3, 2015
@tbosch
Copy link
Copy Markdown
Contributor

tbosch commented Dec 3, 2015

Looks good regarding public api change

@kegluneq kegluneq added the action: merge The PR is ready for merge by the caretaker label Dec 3, 2015
@kegluneq kegluneq assigned jelbourn and unassigned tbosch Dec 3, 2015
@kegluneq
Copy link
Copy Markdown
Author

kegluneq commented Dec 3, 2015

Thanks @tbosch , @jelbourn this is now ready.

@mary-poppins
Copy link
Copy Markdown

Merging PR #5578 on behalf of @jelbourn to branch presubmit-jelbourn-pr-5578.

@kegluneq kegluneq closed this in 0a3a17f Dec 3, 2015
@kegluneq kegluneq deleted the dart_fix_reflection branch December 3, 2015 22:47
@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 type: bug/fix

Projects

None yet

Development

Successfully merging this pull request may close these issues.

APIs do not agree: ReflectionCapabilities & NoReflectionCapabilities

6 participants

X Tutup