node-gcstats icon indicating copy to clipboard operation
node-gcstats copied to clipboard

Publish 1.5.0

Open roikoren755 opened this issue 4 years ago • 15 comments

It seems that v1.5.0 wasn't published to NPM, even though a commit was made to master with [publish npm] in the name of the commit.

Can you please publish this new version?

roikoren755 avatar Jun 23 '20 09:06 roikoren755

👍 100% second this. Maybe merge PR #50 and publish v1.5.1?

baconandon avatar Jul 26 '20 17:07 baconandon

I can second this as well! :ok_hand: It would be great to merge #50 and publish a new version. What do you think @dainis ?

adnanrahic avatar Sep 07 '20 08:09 adnanrahic

+1

thesocialdev avatar Sep 14 '20 16:09 thesocialdev

@roikoren755, @baconandon & @thesocialdev is one of you perhaps interested in forking this repo and maintaining it? It looks like @dainis is no longer around to maintain it. No activity in ~4 months.

otisg avatar Oct 07 '20 00:10 otisg

@otisg TBH I don't have available capacity to carry on with maintenance, but thanks for suggesting a fork.

thesocialdev avatar Oct 13 '20 13:10 thesocialdev

I've forked this repo and audited all dependencies. You can see it on Github, here, or on npm, here.

adnanrahic avatar Oct 21 '20 12:10 adnanrahic

Could you please also get rid of the bundledDependencies? Those should not be necessary anymore: https://github.com/mapbox/node-pre-gyp#configuring

Thanks!

gjasny avatar Oct 21 '20 12:10 gjasny

@gjasny any chance you can submit a PR for that to https://github.com/adnanrahic/node-gcstats ?

otisg avatar Oct 22 '20 02:10 otisg

Hey @gjasny. Could you open a PR in https://github.com/adnanrahic/node-gcstats with that edit and I'll review it ASAP?

adnanrahic avatar Oct 22 '20 08:10 adnanrahic

I'm travelling for the rest of the week. Wont be able to do this before Monday morning.

Thanks, Gregor

On Thu, 22 Oct 2020, 10:48 Adnan Rahić, [email protected] wrote:

Hey @gjasny https://github.com/gjasny. Could you open a PR with that edit and I'll review it ASAP.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/dainis/node-gcstats/issues/49#issuecomment-714336927, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAI4SHQZDLCLVSNIG4W2BYLSL7WU5ANCNFSM4OFPGJLA .

gjasny avatar Oct 22 '20 08:10 gjasny

Thanks @gjasny! There's no rush. I've opened an issue in the fork so we can keep track of it.

Feel free to ping me or add me as a reviewer. :smile: :ok_hand:

adnanrahic avatar Oct 22 '20 09:10 adnanrahic

ping?

orgads avatar Jan 27 '21 21:01 orgads

I'm maintaining a fork of this repo. Anything I can help you with?

adnanrahic avatar Jan 28 '21 08:01 adnanrahic

Can you push a new version to npm?

And can you also publish pre-built binaries for alpine?

orgads avatar Jan 28 '21 08:01 orgads

As I said, I'm maintaining a fork, not this repo.

To install the fork, you can run:

npm install @sematext/gc-stats@latest

I currently don't have any pre-built bins ready. You can open a PR and contribute that if you need it. I'd be happy to review the PR. :smile:

adnanrahic avatar Jan 28 '21 08:01 adnanrahic