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

Create Kind request to rm package from Atmosphere #1

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Create Kind request to rm package from Atmosphere #1

wants to merge 1 commit into from

Conversation

dandv
Copy link

@dandv dandv commented Aug 16, 2015

Hi,

I'm an Atmosphere moderator (@dandv) and found your package https://atmospherejs.com/yee379/livestamp while searching for "livestamp".

Your package has a README identical to the original, and its GitHub link points to the original as well. This may confuse users. The README should describe how this fork is different from the original.

I see that you've added support for epoch. Have you tried submitting a PR to the original maintainer first?

If you only need the patch for a private project, there's no need to publish the modified package to Atmosphere. Simply have the code in a packages directory under your project, and Meteor will pick it up.

What you can do now:

  1. See https://github.com/copleykj/meteor-livestamp/issues/18 and submit the patch to @copleykj
  2. Hide this package from Atmosphere: meteor admin set-unmigrated yee379:livestamp

Thank you for keeping Atmosphere clean,
Dan

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