Skip to content
This repository was archived by the owner on Feb 22, 2018. It is now read-only.

Clean up export. #1313

Closed
wants to merge 1 commit into from
Closed

Clean up export. #1313

wants to merge 1 commit into from

Conversation

BlackHC
Copy link
Contributor

@BlackHC BlackHC commented Aug 5, 2014

ShadowRootAware is defined in annotation.dart and not in annotation_src.dart.

@mhevery
Copy link
Contributor

mhevery commented Aug 5, 2014

Thanks for your contribution! In order for us to be able to accept it, we ask you to sign our CLA (contributor's license agreement). CLA is important for us to be able to avoid legal troubles down the road.

For individuals (a simple click-through form): http://code.google.com/legal/individual-cla-v1.0.html

@mhevery mhevery added cla: no and removed cla: yes labels Aug 5, 2014
@vicb
Copy link
Contributor

vicb commented Aug 5, 2014

LGTM, please amend the commit to follow the guidelines for comments (https://github.com/angular/angular.dart/blob/master/CHANGELOG.md)

@BlackHC
Copy link
Contributor Author

BlackHC commented Aug 5, 2014

I've signed up now (I'm working for Google, too). I'll amend the commit
shortly.
There are issues because of a failing rebase in the g3 tests. Not sure what
to do about that.

On Tue, Aug 5, 2014 at 11:02 AM, Victor Berchet [email protected]
wrote:

LGTM, please amend the commit to follow the guidelines for comments (
https://github.com/angular/angular.dart/blob/master/CHANGELOG.md)


Reply to this email directly or view it on GitHub
#1313 (comment).

@vicb
Copy link
Contributor

vicb commented Aug 5, 2014

@BlackHC this @mhevery is a bot don't worry about the CLA thing. The team will handle the sync with the g3 branch. Thanks.

@BlackHC
Copy link
Contributor Author

BlackHC commented Aug 5, 2014

Okay. I've amended the commit message. Not sure whether it is a fix or a refactor though ^^

ShadowRootAware is defined in annotation.dart and not in annotation_src.dart.
@BlackHC
Copy link
Contributor Author

BlackHC commented Aug 7, 2014

Anything else I need to do to update this request?

Thanks,
Andreas

@chirayuk
Copy link
Contributor

chirayuk commented Aug 8, 2014

@BlackHC – all I need from you is to sign the CLA with the gmail address you used for the commit (visit http://code.google.com/legal/individual-cla-v1.0.html.)  We use an automated test to determine if someone has signed the CLA based on it and this also allows re-verification for future commits.  I'll merge it in shortly after.

@BlackHC
Copy link
Contributor Author

BlackHC commented Aug 8, 2014

Done. It's weird though because I've already signed up in our internal list as well.

Thanks,
Andreas

@chirayuk chirayuk closed this in ed13697 Aug 8, 2014
@chirayuk
Copy link
Contributor

chirayuk commented Aug 8, 2014

Merged.  Thanks for signing the CLA.  I'll mail you @google.com about the CLA list.

chirayuk pushed a commit that referenced this pull request Aug 9, 2014
ShadowRootAware is defined in annotation.dart and not in annotation_src.dart.

Closes #1313
@BlackHC BlackHC deleted the patch-1 branch August 13, 2014 12:34
BlackHC referenced this pull request Oct 9, 2014
Completely remove the deprecated Controller directive.

Major update to the demos to use setRootContext (where applicable) and
to move to Components for the beefier demos (bouncing balls and form).

A temporary work around was required in demos (animation and todo)
so that rootContext have Map interface. The work arounds will disappear
after the context is set to the component.

Library changes were pulled out of the first commit in #1269.

Closes #1401
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Development

Successfully merging this pull request may close these issues.

4 participants