output comments #7
Conversation
|
Wenn du -m benutzt, dann benutze double-quotes, keine single-quotes. Aber am Besten benutzt du einfach kein -m ;) |
|
|
||
| "github.com/jmoiron/sqlx" | ||
| "golang.org/x/sys/unix" |
There was a problem hiding this comment.
Für go1.3 kompatibilität müssen wir offenbar syscall benutzen: https://travis-ci.org/nnev/termine/jobs/102937802 und golang/go#9211
|
Das Meiste sind nits. Wenn du die commit-message editierst kannst du auch gleich squashen und einen commit draus machen statt ≥ 2 :) |
|
|
||
| package main | ||
|
|
||
| func TermWidth() (int, bool) { |
There was a problem hiding this comment.
nit: Benamse die return-parameter. Aus der Signatur+godoc-comment sollte klar sein, was die Funktion zu benutzen ist - in diesem Fall wäre z.B. "(width int, ok bool)" idiomatisch.
|
Ein paar nits, sonst LGTM. Ich überlasse dir ob du die nits fixen willst und dann mergen (das muss nicht nochmal reviewed werden imho), oder ob du einfach direkt mergen willst. :) |
yarpnarp command output comments fixes #3
fixes #3