Skip to content

Added Logentries Driver#27471

Merged
thaJeztah merged 2 commits intomoby:masterfrom
caarlos0:logentries
Oct 18, 2016
Merged

Added Logentries Driver#27471
thaJeztah merged 2 commits intomoby:masterfrom
caarlos0:logentries

Conversation

@caarlos0
Copy link
Contributor

Reopening #22699 because I removed my fork and github lost track of it.

Also made the changes asked by @thaJeztah .

Docs were made on PR docker/docs#244

Signed-off-by: Carlos Alexandro Becker <[email protected]>
Signed-off-by: Carlos Alexandro Becker <[email protected]>
Copy link
Member

@thaJeztah thaJeztah left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks!

ping @vdemeester @albers @sdurrheimer if you could have a quick look at the completion scripts

@albers
Copy link
Member

albers commented Oct 18, 2016

bash completion is perfect 👍

Copy link
Contributor

@sdurrheimer sdurrheimer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ZSH completion LGTM

@thaJeztah
Copy link
Member

Thanks @albers @sdurrheimer !

@thaJeztah thaJeztah merged commit 24582e8 into moby:master Oct 18, 2016
@caarlos0
Copy link
Contributor Author

Thanks all!

@caarlos0 caarlos0 deleted the logentries branch October 18, 2016 10:48
@cpuguy83
Copy link
Member

ping @caarlos0 can you take a look at bsphere/le_go#14 regarding licensing for le_go?

Thanks!

@caarlos0
Copy link
Contributor Author

@cpuguy83 indeed there is no license... I'm not the owner of the lib though, maybe @bsphere can help us here :)

@bsphere
Copy link

bsphere commented Feb 15, 2017

@cpuguy83 @caarlos0 just added a LICENSE file to that repo (MIT).
let me know if you have any issues with such license

@cpuguy83
Copy link
Member

❤️ thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants