TP Honey

First off, its is soo awesome. So i gave it a test drive 🚗 , and when i tried it in my default shell zsh. It did not work. Once...

Can you please rebase, this is why travisCI maybe failing for this PR. I tested this with #168 apologies for the inconvenience.

Hi @ITBlogger thanks for the rebase. The tests look to be passing again 👍 So i spent a little more time reviewing the PR. There are a few things that...

The addition of Variant, will make the module incompatible with people using puppet 3. We cannot merge this PR until then, apologies. We have been working to migrate modules fully...

I had a look at the change, and this seems to be additive so it shouldnt affect any existing functionality. I will defer to Brad, as i have not looked...

Have you had a look at implementing @bradrydzewski 's suggested changes and rebasing of master.

Hi @tlinhart thanks for the PR, to start with. Just to confirm your intentions, this is to enable pushing the drone-docker plugin to ECR. If you need images mirrored you...

So for anyone else that gets here, i added in readonly mounts of the drives i was interested in. how i configured it with docker was, in my docker-compose for...

It looks like the user you are using to run the runner, does not have permissions to change docker settings. specifically the sock file. Could you also please explain what...

Hey @kylewallpe from looking at the logic of the plugin, if autotag is true, then user set tags are ignored. [drone-manifest/main.go at 92722b712a4a378262ca8f3e39e34248646c8a7b · drone-plugins/drone-manifest · GitHub](https://github.com/drone-plugins/drone-manifest/blob/92722b712a4a378262ca8f3e39e34248646c8a7b/main.go#L207-L209) Can you confirm