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

Misc minor changes #1061

Closed
wants to merge 5 commits into from
Closed

Misc minor changes #1061

wants to merge 5 commits into from

Conversation

vicb
Copy link
Contributor

@vicb vicb commented May 23, 2014

@jbdeboer could you please review this PR when you have time (low priority).

It's composed of multiple little commit to ease cherry-picking (/rejecting) some if needed - the second commit is style change only.

Check the commit comment for detailed information.

vicb added 5 commits May 23, 2014 11:00
- Add some doc,
- Add types,
- Make AnnotationsMap._map private,
- Simplify AnnotationsMap.annotationsFor
- Make _fieldMetadataCache a static final member,
- Simplify some code & make some methods private,
- Add docs
@vicb vicb added cla: yes and removed cla: no labels May 23, 2014
@mvuksano mvuksano assigned vicb and unassigned vicb May 26, 2014
@vicb vicb added this to the 0.14.0 milestone Jun 25, 2014
@mhevery
Copy link
Contributor

mhevery commented Jul 25, 2014

Could you rebase this on master, to make the merge easier?

vicb added a commit to vicb/angular.dart that referenced this pull request Jul 31, 2014
vicb added a commit to vicb/angular.dart that referenced this pull request Jul 31, 2014
@vicb vicb closed this in 65fc117 Jul 31, 2014
@vicb vicb deleted the 0523-misc branch July 31, 2014 12:56
vicb added a commit that referenced this pull request Aug 5, 2014
dsalsbury pushed a commit to dsalsbury/angular.dart that referenced this pull request Aug 7, 2014
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.

2 participants