mediasuitenz / mappy

A configurable map engine
MIT License
3 stars 2 forks source link

Fix/es6 transpilation #63

Closed digitalsadhu closed 9 years ago

digitalsadhu commented 9 years ago

Changes to package.json

Changes to testem setup

digitalsadhu commented 9 years ago

@markstuart @matt-in-a-hat Review pls when you get a chance

markstuart commented 9 years ago

What was your reasoning for fixed dependencies?

digitalsadhu commented 9 years ago

My thinking was that explicit updates means you go through once in a while update all the dependencies, run tests, commit and publish a new version. Means you are forced to be more purposeful with updates. I left dev dependencies dynamic. 

What do you think?

On Sun, Jan 18, 2015 at 3:57 PM, Mark Stuart notifications@github.com wrote:

What was your reasoning for fixed dependencies?

Reply to this email directly or view it on GitHub: https://github.com/mediasuitenz/mappy/pull/63#issuecomment-70394492

markstuart commented 9 years ago

I actually like this. Probably means we don't need to shrink wrap before releases. We should probably let our contributors know so that they don't worry about the shrink wrap file. Do you want remove the file in this PR?

digitalsadhu commented 9 years ago

Can do. Thinking about it, maybe we should fix the dev deps as well?

On Sun, Jan 18, 2015 at 8:32 PM, Mark Stuart notifications@github.com wrote:

I actually like this. Probably means we don't need to shrink wrap before releases. We should probably let our contributors know so that they don't worry about the shrink wrap file.

Do you want remove the file in this PR?

Reply to this email directly or view it on GitHub: https://github.com/mediasuitenz/mappy/pull/63#issuecomment-70399276

markstuart commented 9 years ago

Yep, sounds good, keep the contributors test libs in line.

markstuart commented 9 years ago

@digitalsadhu, take a look at this when you get a chance please. Lets get this live!

digitalsadhu commented 9 years ago

:+1: