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.

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

Request: Break Velocity Into Modules #4

Closed
chchrist opened this issue Apr 29, 2014 · 6 comments
Closed

Request: Break Velocity Into Modules #4

chchrist opened this issue Apr 29, 2014 · 6 comments

Comments

@chchrist
Copy link

Hello
Now velocity is a single big file and it is difficult for many devs to work on it. Could you break it to multiple modules (commonJS, AMD)?

Cheers

@julianshapiro julianshapiro changed the title Break the plugin into modules Break Velocity Into Modules Apr 29, 2014
@kc-dot-io
Copy link

👍

@julianshapiro consider component.io for modularity. At worst you should have a component.json file, but if you plan to break it out further, component makes for a pretty robust hierarchy. We use it religiously. Happy to run you through it in our next meet up.

@jtangelder
Copy link

A simple way how i like to work is just to simply concat the files. So you don't get the overhead of the umd tools. Did this for Hammer.js and am still happy with it :-)

@julianshapiro
Copy link
Owner

Jorik,

First off, Hammer is AWESOME.

Second, I actually prefer your suggestion over the others. I will start doing this beginning this weekend.

Meanwhile, I'm also going to take Kyle up on his dinner offer to hear more about component :)

@julianshapiro julianshapiro changed the title Break Velocity Into Modules Request: Break Velocity Into Modules May 1, 2014
@jtangelder
Copy link

I'm glad to hear that! If you like, you can take a look at the gruntfile I use for Hammer, contains some basic logic that could be useful. Also i find jshint and jscs very useful for consistent code when people are going to contribute.

@fritx
Copy link

fritx commented May 4, 2014

Personally, I recommend gulp instead of grunt for velocity ;)

@julianshapiro
Copy link
Owner

This will be happening once we drop the jQuery dependency. Thanks for the suggestions, guys!

We'll be using @jtangelder's approach to grunt concatenation.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants