Skip to content

Conversation

@dandv
Copy link

@dandv dandv commented Nov 29, 2014

Hi David,

I'm a Meteor.js dev and volunteer package maintainer for Meteor. Isotope is a great layout library, and adding it to Meteor's ecosystem will expose it to many new devs (Meteor has 20k+ stars on GitHub), likely bringing in new testers and contributors.

This PR will let you directly publish updated versions of the library as they become available. All you have to do is create an account at https://meteor.com/ (click SIGN IN, then Create account). After you've done that, please let me know the name of the account, and I'll add you as a maintainer.

I've already published the current version of the package on Atmosphere (Meteor's package directory). When you release new versions, you'll be able to publish them to Atmosphere by running

grunt meteor

Thanks & best regards,
Dan, official Meteor integrations team

@dandv dandv force-pushed the meteor-integration branch from f99d0ca to 9e5c89b Compare November 29, 2014 12:14
@desandro
Copy link
Member

Hi Dan! Thanks so much for this PR. I appreciate you taking care of everything for me :)

I'd like to keep this PR open for a while to see how much interest there is in Meteor support. Please +1 this issue if you would like to see Meteor support for Isotope.


I'm hesitant to add support for a couple reasons

  • In order to support Meteor I have to add a bunch of extra stuff. Having so many Meteor-specific feels like adding extra clutter. (I feel like publish.sh and runtests.sh are better served as global scripts that should be generalized for all Meteor packages, rather than specific for each project.)
  • Another ecosystem to keep track of. Isotope already supports Bower, npm, Browserify, UMD.
  • There has been no previous requests for Meteor support in three years.

@wuxianliang
Copy link

+1

@wuxianliang
Copy link

with packery layout mode in one package is better.

@dandv
Copy link
Author

dandv commented Nov 30, 2014

Hi David! Thanks for prodding me to get rid of those .sh files :) Indeed they should be generalized to the process, and I've moved them into a grunt file patch.

As for lack of requests for Isotope & Meteor, that's because we've had a few wrapper packages, that have been getting out of date. One that's still kicking is here. You can see 30 app installs, and that's only since around October, when we started tracking install counts.

As for Meteor being a different package system, that is an acknowledged issue, and there are efforts to update Meteor's packages repository automatically from GitHub. That will take quite a bit of work though, and in the meanwhile I was hoping that we could provide packaging straight from the source. I do understand your concerns though, and I'll leave this up to you.

Best regards,
Dan

@physiocoder
Copy link

+1

hopefully even for v1.5

@fpaboim
Copy link

fpaboim commented Dec 30, 2014

+1

4 similar comments
@QWp6t
Copy link

QWp6t commented Dec 30, 2014

👍

@silverbuggy
Copy link

+1

@superbilk
Copy link

+1

@lagden
Copy link

lagden commented Feb 18, 2015

+1

@rdibona
Copy link

rdibona commented Mar 9, 2015

+1 and I'm more than happy to pay for the commercial license.

@dnbkr
Copy link

dnbkr commented Jul 6, 2015

+1 and also worth mentioning that as of now, the current package on atmosphere is at 1,780 installs

@cmaddalozzo
Copy link

+1

2 similar comments
@dcworldwide
Copy link

+1

@Vheissu
Copy link

Vheissu commented Feb 17, 2016

+1

@timneutkens
Copy link

@dandv @desandro this pull request can be closed in favor of npm support in Meteor right?

@dandv
Copy link
Author

dandv commented Sep 12, 2016

Correct.

@dandv dandv closed this Sep 12, 2016
@dandv dandv deleted the meteor-integration branch September 12, 2016 16:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.