Merge pidtree back into ps-tree?

[simonepri]

Following this old issue:
#10

Who are the current maintainers of this package?
For what I saw:
@indexzero is the author (can push on the repo and can publish on npm)
@nelsonic is one of the maintainers (can only push on the repo)
Am I missing someone?

Can we add more contributors to this repo?
#18 (comment)

cc @zixia @soyuka

[soyuka]

We maybe could release a community fork? I've a pending fix on @zixia 's fork also haha

[simonepri]

Let's see what @nelsonic says about that. But this package has 4 Mln downloads per month so if we can keep working on this repo would be better for everyone.

Actually the only one that now can push on npm is @indexzero and he have said that he don't want to maintain this anymore.

@soyuka are you referring to this https://github.com/zixia/ps-tree/pull/1?
Can you create the PR on this repo too?

[soyuka]

Can you create the PR on this repo too?

It's based on @zixia ´s work on the repository so I guess we need to merge his changes first :|

[simonepri]

Oh right!
So let's make a single PR with both changes.

I missed that because @zixia has done all the work in this single PR.
#17

[huan]

@soyuka Let me merge your change in my PR, then I believe my PR at here will include your PR....

:-/

[simonepri]

@zixia can you open another PR removing the addition of appveyor and the changes of the readme?
We have already #29 for that.

[huan]

@simonepri Ok no problem.

I had just modified my PR and remove the README&Appveyor changes.

Please let me know if you think I should modify other parts as well.

[simonepri]

Thank you! I've commentend under the PR.

[simonepri]

@indexzero See also this
#28 (comment)

[soyuka]

FTR merged #17 #24 #27 and #28 to https://github.com/soyuka/ps-tree

[simonepri]

@soyuka Lets wait @nelsonic before proceed please.

I don't see any advantage of working on a community fork if we can push on npm from this one.

[soyuka]

@simonepri I don't want to bother anyone ;) I'm just using this in pidusage-tree, and therefore will use my fork for the time being :). Obviously if things gets handled here I'll use the original version.

https://twitter.com/andrestaltz/status/961600427832872961

TL;DR

• Publish forked packages
• Inform, don't annoy, the original author
• Open source is pointless unless for reading the source
• Less "githubness", more openness
• Small libraries!
• Disagreement is okay, it's great actually
• Minimize consensus surface

[simonepri]

@nelsonic @indexzero news?

[soyuka]

@simonepri pinging them is useless just let them answer if they want to I'm sure they've been notified by now ;). Note that my last contributions to this were almost 1 year ago and there has been no answer yet.

[simonepri]

Actually nelsonic has already merged some of my PRs some days ago:
#26
#25

So I don't think is useless. They are alive. 😁