elliotstokes / gpx-parse

GPX parser
MIT License
44 stars 40 forks source link

Browserfiying the module #8

Closed Wilkins closed 9 years ago

Wilkins commented 9 years ago

Adding the npm module, the Grunt config, and a mockup of the lib-cov module to be able to browserify without errors. Everything seems OK.

elliotstokes commented 9 years ago

ah also need to remove the gpx-parse file from the commit. Might be worth changing the destination to be gpx-parse-browser or something as well.

Wilkins commented 9 years ago

You want to remove the browserified file from the repository ? Or juste change the name ?

elliotstokes commented 9 years ago

I think probably remove it as its a generated file.

elliotstokes commented 9 years ago

Unless you have a reason why you want to add it?

Wilkins commented 9 years ago

Well, yeah I wish it was already included in the repository. So this repository could be included as a git submodule in other projects, and be used out of the box. For example, in the project I'm working on, I use gpx-parse as a submodule. I want the module to be pulled when I clone the repository, and I want it to be included in my javascript when I run Grunt, without having to know how to browserify this particular repository. Each repository should be built already. See some examples:

This is my opinion, of course this is your repository, your choice :-)

elliotstokes commented 9 years ago

How about we put it in a folder called dist or distribution?

Wilkins commented 9 years ago

Perfect for me.

elliotstokes commented 9 years ago

Im not sure I can edit this push so are you able to make the modification to the grunt script to build to a dist folder and commit the browserify file into that folder instead? Ill merge it all then.

Thanks a lot.

Wilkins commented 9 years ago

Yeah no problem, I'll do theses modifications in few hours. I'm closing this PR, I will open a new one.

elliotstokes commented 9 years ago

No probs. Ill check back later. Thanks again