Skip to content
This repository was archived by the owner on Jan 26, 2018. It is now read-only.

Conversation

@marcinwyszynski
Copy link
Contributor

This is an attempt to fully address #3

Hope this makes sense @bradrydzewski. If not, please point me in the right direction.

@bradrydzewski
Copy link
Member

thanks, LGTM :)

@marcinwyszynski
Copy link
Contributor Author

How about a merge then?

@bradrydzewski
Copy link
Member

@marcinwyszynski I just need to pull down and run some tests and will then get it merged right away. thanks!

@bradrydzewski
Copy link
Member

bradrydzewski commented Sep 5, 2016

one more minor request ... could you please bump the docker image tag in the .drone.yml to 1.1.0

  docker:
    repo: lgtm/lgtm
-   tag: [latest, 1.0.0]
+   tag: [latest, 1.1.0]
    when:
      branch: master
      event: push

thanks!

@marcinwyszynski
Copy link
Contributor Author

@bradrydzewski No bother, done.

@albertsj1
Copy link

Seems like this should be merged already.

@marioizquierdo
Copy link

Are we there yet?

@marcinwyszynski
Copy link
Contributor Author

@marioizquierdo if you're asking me - my work here is done. If you're asking @bradrydzewski he does not seem to frequent these premises any more, so feel free to use my fork.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants