This repository has been archived by the owner. It is now read-only.

chore(demo): move all demos into a single folder #688

Closed
wants to merge 1 commit into
from

Conversation

Projects
None yet
3 participants
@vicb
Contributor

vicb commented Mar 6, 2014

fixes #624

@vicb vicb added cla: yes and removed cla: no labels Mar 6, 2014

@bgourlie

This comment has been minimized.

Show comment Hide comment
@bgourlie

bgourlie Mar 6, 2014

Contributor

I agree with @shailen 's comment in #624. Moving all demos into one folder would be encouraging bad practice.

Contributor

bgourlie commented Mar 6, 2014

I agree with @shailen 's comment in #624. Moving all demos into one folder would be encouraging bad practice.

@mhevery

This comment has been minimized.

Show comment Hide comment
@mhevery

mhevery Mar 7, 2014

Contributor

I think the folder should be renamed to test_app rather than demo. I think it would better reflect the purpose of the folder. Maybe add some routing to it.

I need a single app where I can test out the syntax/workings of the system.

Contributor

mhevery commented Mar 7, 2014

I think the folder should be renamed to test_app rather than demo. I think it would better reflect the purpose of the folder. Maybe add some routing to it.

I need a single app where I can test out the syntax/workings of the system.

mhevery added a commit to mhevery/angular.dart that referenced this pull request Mar 7, 2014

mhevery added a commit to mhevery/angular.dart that referenced this pull request Mar 7, 2014

mhevery added a commit to mhevery/angular.dart that referenced this pull request Mar 7, 2014

@vicb vicb closed this in 1ca6d17 Mar 7, 2014

@vicb vicb deleted the vicb:demos branch Mar 14, 2014

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