Mikhail Dyakov eicca

Organizations

@brainly
eicca starred relops/cqlc
@eicca
eicca starred gocql/gocql
eicca created repository eicca/smart-translate-server
eicca commented on pull request alaa/pencil-go#12
@eicca

:+1:

eicca commented on pull request alaa/pencil-go#8
@eicca

:+1:

eicca commented on pull request alaa/pencil-go#8
@eicca

As I remember we agreed that the project should already contain Godeps. Should we make this decision consistent?

eicca commented on pull request alaa/pencil-go#8
@eicca

different path for main repository: alaa/pencil-go

eicca commented on pull request alaa/pencil-go#7
@eicca

:+1:

eicca commented on pull request alaa/pencil-go#7
@eicca

Just my opinion: I would return the error here and would write log on the top level. In this case it would be free of side effects. It also could m…

eicca commented on pull request alaa/pencil-go#7
@eicca

I think we can just use nil as nil value for slices.

eicca commented on pull request alaa/pencil-go#2
@eicca

What do you think about checking the error?

eicca commented on pull request alaa/pencil-go#2
@eicca

It looks a bit strange that we just need to proxy this method. I have a feeling that we're overcomplicating the stuff. As I understand we're just c…

eicca commented on pull request alaa/pencil-go#2
@eicca

What does this TODO mean?

eicca commented on pull request alaa/pencil-go#2
@eicca

Just my personal opinion, but it feels a bit over engineered. Since the docker.Client is not an interface we're wrapping it around with another cli…

eicca commented on commit alaa/pencil-go@e8a19edcd5
@eicca

I would return an error here. log.Fatal should be called only in extreme case. Here the container could be removed, or something like this, which i…

eicca commented on commit alaa/pencil-go@e8a19edcd5
@eicca

Would just return d.client.ListContainers(options) work?

eicca commented on commit alaa/pencil-go@21df9b978f
eicca created branch master at eicca/reframe-test
eicca created repository eicca/reframe-test