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

Meteor Package support #17

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from

Conversation

gadicc
Copy link

@gadicc gadicc commented Jan 27, 2015

Hey! To continue the discussion from gadicc/meteor-famous-views#186, I'm back from holidays and have reworked our code to use your library instead of using MutationObservers like we had before (thanks!).

The best way to do this is to have your code available as a Meteor package. Lately there has been a big effort in the community to get offical support from package authors directly rather than having a community member maintain a wrapper package that they need to keep up to date.

As per those guidelines, I'm submitting this PR for your consideration and have already registered "sdecima" as a meteor organization to expedite the process & testing. All you have to do is create an account at https://meteor.com/ (click SIGN IN, then Create account; unfortunately with a different username). After you've done that, please let me know the name of the account, and I'll add you as a maintainer. The package appears on Atmosphere thus: https://atmospherejs.com/sdecima/javascript-detect-element-resize.

Lastly, there is a script, meteor/publish.sh, which you can run the same time as npm publish (other authors have this integrated as part of their grunt-file / Makefile). Alternatively, the community is hard at work on a solution where on every push to github, the Meteor package is automatically built on and published from the cloud.

Sorry for the long explanation and hope this all works for you :)

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

Successfully merging this pull request may close these issues.

1 participant