-
Notifications
You must be signed in to change notification settings - Fork 46
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
Updating the npm node #20
Comments
Just wanted to chime in support and say I'd love to use this module from npm, so it would be great to have it be up to date! It looks like the owner of this repo hasn't had a lot of activity recently, so a maintained fork may be best. |
Damn! I don't see this issue ... My bad .. |
If someone is always ready to publish in npm and/or maintain the repo I can give him access. Just tell me (and eventually insist if I did not respond) |
Maintaining our own npm package at this point: https://www.npmjs.com/package/easybotics-rpi-rgb-led-matrix I'm totally willing to polish it up a bit, make a pull request and maintain your npm repo in the near future if you want. |
Perhaps the best move are:
What do you think about it ? |
Probably best not to delete anything; for the people who might rely on older versions of this repo. But a 'not maintained' message probably works. I can be added as a owner of the npm, and bring it inline with a working version of your repo. The way I see it, the only issue with the npm package is that its 3 years old, not really essential that it receives whatever new features that are in my fork. So both can coexist once its updated. |
I had a link to your repo in readme. |
thanks, I updated the npm |
https://www.npmjs.com/package/node-rpi-rgb-led-matrix
is still tracking the 3 year old version,
I plan on releasing some node modules pulling the new 64x64 enabled library as an npm dependency in the near future; and will be actively maintaining my fork as well.
I could either put my fork up as a new npm package, or could become a maintainer of this version on github/npm if you're up for it.
The text was updated successfully, but these errors were encountered: