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

Send status on PR open #47

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Send status on PR open #47

wants to merge 3 commits into from

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