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

Offering to maintain nps #234

Open
mf-bruce opened this issue Apr 28, 2022 · 6 comments
Open

Offering to maintain nps #234

mf-bruce opened this issue Apr 28, 2022 · 6 comments

Comments

@mf-bruce
Copy link

First off, thanks @sezna for building nps in the first place! We have been using it at Markforged for some time now, and have found it to be elegant in its simplicity, whilst providing just enough functionality to be incredibly powerful!

It seems that nps has lacked maintenance for some years now, there are outstanding security issues, and feature requests such as positional args (something we have implemented in our fork).

We would like to offer to maintain the nps package, so opening this issue to start that conversation!

@sezna
Copy link
Owner

sezna commented Apr 28, 2022

Yeah, I took over because my previous job was using it and it wasn't being maintained. Now, I'm no longer in that role and I haven't had the time to give it any love. Happy to find somebody who is willing to give it the love it needs!

@mf-bruce
Copy link
Author

Awesome! How do we go about transferring this? Would it make sense to transfer the npm package ownership so we can publish from our repo moving forward?

@Melmoth-the-Wanderer
Copy link

@mf-bruce have you taken over the project?

@mf-bruce
Copy link
Author

mf-bruce commented Jun 9, 2022

Yep, you can find our fork

Would be nice to transfer the original package name on npm, or at least deprecate it. FYI it's possible the npm deploy key for the original package was leaked via travis:( https://arstechnica.com/information-technology/2021/09/travis-ci-flaw-exposed-secrets-for-thousands-of-open-source-projects/

@mf-bruce
Copy link
Author

@sezna the current status quo will work fine for folks who proactively want to use a newer version of nps but there are some things we can do to smooth the transition for users who just have "latest" specified in package.json, or new folks discovering this package. Raising these here as I will need some assistance from yourself if we are going to execute on them.

  • Transfer issues to our fork
    ** I would need write access to this repo to transfer issues)
  • Set our fork as the root repo for other forks
    ** I asked github support about this and they need you to additionally submit a ticket which references ours
  • Archive this fork
  • Transfer ownership of the nps npm package to us

@nareshbhatia
Copy link

nareshbhatia commented Aug 9, 2022

I stumbled upon this package today and found it very interesting. But looking at the last commit almost 2 years ago, was hesitant to pick it up. Until I saw this issue. Great job @mf-bruce to keep it going.

I would personally like to see mf-bruce's fork merged over here so that star ratings can be maintained. There is a lot of equity build into the stars and the downloads - would be sad to loose it all. Also, there is the complication of moving the issues and all. @Senza, in the best interest of the community, please consider merging mf-bruce's fork here and giving him permissions to publish on npm. You both can co-own the package (or may be create an org for it).

P.S. I also see @kentcdodds on the contributor's list, so this has to be good :-)

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

4 participants