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

Add @jeromew as npm package owner #55

Closed
brianc opened this issue Aug 22, 2016 · 8 comments
Closed

Add @jeromew as npm package owner #55

brianc opened this issue Aug 22, 2016 · 8 comments

Comments

@brianc
Copy link
Owner

brianc commented Aug 22, 2016

Hey @jeromew - if you'd like you can submit a PR with npm name added as a package owner. I can then publish to npm and after that both of us can publish!

@jeromew
Copy link
Collaborator

jeromew commented Aug 22, 2016

@brianc thanks ! I will do that shortly after merging the PRs

@jeromew
Copy link
Collaborator

jeromew commented Aug 22, 2016

@brianc I am not sure what I am supposed to do after reading the link you provided.

npm owner add jeromew pg-copy-streams

gave me

you do not have permission to publish "pg-copy-streams". Are you logged in as the correct user? : pg-copy-streams

I thought it would maybe update package.json but it did nothing.

@brianc
Copy link
Owner Author

brianc commented Aug 22, 2016

ah i can do it - i really just needed your npm name. Wasn't sure if it was the same as your github name. 😄

@jeromew
Copy link
Collaborator

jeromew commented Aug 22, 2016

npm whoami => jeromew

@brianc
Copy link
Owner Author

brianc commented Aug 22, 2016

okay I added you!

@jeromew
Copy link
Collaborator

jeromew commented Aug 22, 2016

thanks. It seems to have worked. I am going to try and publish a new minor version with the recent PRs.

@brianc
Copy link
Owner Author

brianc commented Aug 22, 2016

awesome! you should be able to just run make minor in the project dir

https://github.com/brianc/node-pg-copy-streams/blob/master/Makefile#L11

On Mon, Aug 22, 2016 at 11:23 AM, jeromew notifications@github.com wrote:

thanks. It seems to have worked. I am going to try and publish a new minor
version with the recent PRs.


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#55 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AADDoRPwLUKgo5zHmV2cVlv44mKDRd_nks5qiczlgaJpZM4JqAXE
.

@jeromew
Copy link
Collaborator

jeromew commented Aug 22, 2016

Done !

@jeromew jeromew closed this as completed Aug 22, 2016
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

2 participants