Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Use Array instead of List #3514

Closed
alexeagle opened this issue Aug 6, 2015 · 5 comments
Closed

Use Array instead of List #3514

alexeagle opened this issue Aug 6, 2015 · 5 comments
Assignees
@mhevery
Labels
breaking changes effort1: hours help wanted An issue that is suitable for a community contributor (based on its complexity/scope). refactoring Issue that involves refactoring or code-cleanup
Milestone
alpha-36

Comments

@alexeagle
Copy link
Contributor

And maybe always use the short syntax like number[]
@alexeagle alexeagle self-assigned this Aug 6, 2015
@alexeagle alexeagle added this to the alpha-35 milestone Aug 6, 2015
@mhevery mhevery added comp: core effort1: hours help wanted An issue that is suitable for a community contributor (based on its complexity/scope). refactoring Issue that involves refactoring or code-cleanup labels Aug 6, 2015
@IgorMinar IgorMinar modified the milestones: alpha-36, alpha-35 Aug 21, 2015
sebholstein added a commit to sebholstein/angular that referenced this issue Aug 24, 2015
@sebholstein
chore(typescript): use shorthand array syntax
6b96b67 
closes angular#3514
@alexeagle alexeagle removed their assignment Aug 27, 2015
@alexeagle
Copy link
Contributor Author

I'm not currently working on this, it would be a good community contribution.

@mhevery mhevery assigned mhchen and mhevery and unassigned mhchen Aug 28, 2015
mhevery added a commit to mhevery/angular that referenced this issue Aug 28, 2015
@mhevery
chore(ts2dart): replace List with Array
6639954 
Closes angular#3514
mhevery added a commit to mhevery/angular that referenced this issue Aug 28, 2015
@mhevery
chore(ts2dart): replace List with Array
6059c37 
Closes angular#3514
mhevery added a commit to mhevery/angular that referenced this issue Aug 29, 2015
@mhevery
chore(ts2dart): replace List with Array
892cca2 
Closes angular#3514
mhevery added a commit to mhevery/angular that referenced this issue Aug 29, 2015
@mhevery
chore(ts2dart): replace List with Array
d2fed23 
Closes angular#3514
@yanivefraim
Copy link

@alexeagle - I would love to help, need some help starting with this. 10x!

@alexeagle
Copy link
Contributor Author

Misko is already working on it, sorry!

@mhevery
Copy link
Contributor

mhevery commented Aug 31, 2015

FYI: Part of this work: https://github.com/angular/angular/pull/3889/commits

mhevery added a commit to mhevery/angular that referenced this issue Aug 31, 2015
@mhevery
chore(ts2dart): replace List with Array
72ab057 
Closes angular#3514
mhevery added a commit to mhevery/angular that referenced this issue Aug 31, 2015
@mhevery
chore(ts2dart): replace List with Array
8ae8024 
Closes angular#3514
itslenny pushed a commit to itslenny/angular that referenced this issue Aug 31, 2015
@mhevery @itslenny
chore(ts2dart): replace List with Array
5dcef7a 
Closes angular#3514
@angular-automatic-lock-bot
Copy link

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.
Assignees

@mhevery mhevery

Labels
breaking changes effort1: hours help wanted An issue that is suitable for a community contributor (based on its complexity/scope). refactoring Issue that involves refactoring or code-cleanup
Projects
None yet
Milestone
alpha-36
Development

No branches or pull requests
5 participants
@alexeagle @mhevery @IgorMinar @mhchen @yanivefraim