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

Create new npm package #5

Merged
merged 2 commits into from Oct 19, 2016
Merged

Create new npm package #5

merged 2 commits into from Oct 19, 2016

Conversation

@Shwetajain148
Copy link

@Shwetajain148 Shwetajain148 commented Oct 7, 2016

As we have to create new npm package. Below are the changes that are done in this PR.

-update package version
-update author details
-update README.md reference

Created Package link: loggly-winston-bulk.

-rename package name to 'loggly-bulk'
-update package version to 1.0.4 as it is new package
-add loggly, winston keywords
-update author details
@@ -81,7 +81,7 @@ Once you have valid configuration and credentials you can run tests with [npm][7
npm test
```

#### Author: [Charlie Robbins](http://blog.nodejitsu.com)
#### Author: [Shweta Jain](https://www.loggly.com/)

This comment has been minimized.

@mchaudhary

mchaudhary Oct 15, 2016

Should we change name ? @mostlyjason thoughts ?

This comment has been minimized.

@Shwetajain148

Shwetajain148 Oct 17, 2016
Author

@mostlyjason Please let me know your views on this.

This comment has been minimized.

@mostlyjason

mostlyjason Oct 17, 2016

Sure we can update it or you can add a 2nd line

This comment has been minimized.

@Shwetajain148

Shwetajain148 Oct 19, 2016
Author

@mchaudhary @mostlyjason I have made changes in the README.md file as per our discussion on author name. Please review..

"description": "A Loggly transport for winston",
"author": "Charlie Robbins <charlie.robbins@gmail.com>",
"author": "Shweta Jain <sjain@loggly.com>",

This comment has been minimized.

@mchaudhary mchaudhary merged commit 6c91cb5 into loggly:master Oct 19, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants
You can’t perform that action at this time.