Skip to content

/ loggly-jslogger

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

Jump to bottom

Add commonjs / npm support #31

Merged
merged 1 commit into from Sep 18, 2015
Merged

Add commonjs / npm support #31

merged 1 commit into from Sep 18, 2015

Conversation

@mattlewis92
Copy link
Contributor

@mattlewis92 mattlewis92 commented Sep 15, 2015

Closes #30
Matt Lewis
Add commonjs / npm support
2e0bb96
mostlyjason pushed a commit that referenced this pull request Sep 18, 2015
Jason Skowronski
Merge pull request #31 from mattlewis92/master
e613f77 
Add commonjs / npm support
@mostlyjason mostlyjason merged commit e613f77 into loggly:master Sep 18, 2015
@mattlewis92
Copy link
Contributor Author

@mattlewis92 mattlewis92 commented Sep 18, 2015

Great, thanks a lot!

On 18 Sep 2015, at 19:21, Jason Skowronski notifications@github.com wrote:

Merged #31.

—
Reply to this email directly or view it on GitHub.
@mattlewis92
Copy link
Contributor Author

@mattlewis92 mattlewis92 commented Oct 12, 2015

Any chance this could be published to npm as well please? (npm publish from the root of the repo) Thanks!
@mostlyjason
Copy link
Contributor

@mostlyjason mostlyjason commented Nov 17, 2015

@psquickitjayant can you do this?
@kpdecker
Copy link

@kpdecker kpdecker commented Feb 23, 2016

I've published this under https://www.npmjs.com/package/loggly-jslogger since I needed this. Glad to transfer ownership to someone actually in the project as I'm not certain that I can maintain this, not being closely tied to the project.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants
@mattlewis92 @mostlyjason @kpdecker
You can’t perform that action at this time.