search

nodejs / llnode

An lldb plugin for Node.js and V8, which enables inspection of JavaScript states for insights into Node.js processes and their core dumps.
Other
1.15k stars 100 forks source link

Give npm collaborator access to @mmarchini ? #234

Closed joyeecheung closed 6 years ago

joyeecheung commented 6 years ago

As the title says.

joyeecheung commented 6 years ago

What do other collaborators think? (There isn't a team to ping so @indutny @cjihrig @bnoordhuis )

cjihrig commented 6 years ago

Makes sense to me.

cjihrig commented 6 years ago

Not to sidetrack this issue, but do rnchamberlain and hhellyer still require npm publish rights?

joyeecheung commented 6 years ago

Oh right I think I forgot to ping other people who have npm collaborator access right now, sorry. So here it is: @rnchamberlain @hhellyer

joyeecheung commented 6 years ago

I think we have left this issue open for enough amount of time.

Judging from https://www.npmjs.com/package/linux-perf I invited @mmarchini to the npm collaborators (let me know if I messed up).

mmarchini commented 6 years ago

I think it worked. Thanks!

richardlau commented 6 years ago

I think we have left this issue open for enough amount of time.

Judging from https://www.npmjs.com/package/linux-perf I invited @mmarchini to the npm collaborators (let me know if I messed up).

Wasn't this for https://www.npmjs.com/package/llnode?

joyeecheung commented 6 years ago

@richardlau I meant judging from that the npm handle @mmarchini is releasing a package under @mmarchini 's github I believe that npm handle also belongs to @mmarchini (instead of somebody who happens to be using the same handle)

joyeecheung commented 6 years ago

Also I believe this can be closed now!