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

Optimist version #1

Closed
florianf opened this issue Jan 31, 2016 · 2 comments
Closed

Optimist version #1

florianf opened this issue Jan 31, 2016 · 2 comments

Comments

@florianf
Copy link

Hi,

thanks to your work, my TileMill for TileOven (https://github.com/florianf/tileoven) now works on Node 4. I also cherry picked some patches of your branch, thanks for that ;-)

Could you please update this package.json to also require optimist 0.6.0, this would fix TileMill test?

Regards,
Florian

@paulovieira
Copy link
Owner

Fantastic! I would never though someone out there would use this stuff :-)

I changed the names of the modules/repos to make them more explicit:

  • paulovieira/tilelive-climais now paulovieira/tilelive (and package.json as optimistic @0.6)
  • paulovieira/millstone-clima is now paulovieira/millstone
  • paulovieira/bones-clima is now paulovieira/bones

Now they are also proper forks (with a reference to the original repo), which is useful to see the changes done to the original code.

I'll submit a pr to tileoven with these changes.

@pvml
Copy link

pvml commented Feb 1, 2016

Also, note that in the paulovieira/bones repo we must use the 1.3 branch. So it should be paulovieira/bones#1.3

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

3 participants