и че не на гитхабе?

на гихабе-то ревью делать удобнее

я туда не заливал еще @onokonem

самое время тогда!

@onokonem https://github.com/therealgofman/smsd только не ругайтесь, я не совсем программист :)

я бы советовал начать с прогона go fmt

потом - go imports

db и logs в .gitignore

потом - gometalinter

вот это - криминал: https://github.com/therealgofman/smsd/blob/master/smsd.go#L4 импортить надо всегда по полному пути

как сделаете fmt и imports - пишите

Ок

gometalinter тоже прогоните, ок?

go fmt и go imports хорошо в редакторе на сохранение повесить

ну, в vscode оно так по-умолчанию

оно везде так по умолчанию

Раз такая пянка, то посмотрите ещё на мой маленький проект, пожалуйста :) https://github.com/alexsadliak/gpm-stats

Раз такая пянка, то посмотрите ещё на мой маленький проект, пожалуйста :) https://github.com/alexsadliak/gpm-stats

flags умеет сам фориировать usage, и лучше на это полагаться - не придется синхронизировать текст и флаги потом

flags умеет сам фориировать usage, и лучше на это полагаться - не придется синхронизировать текст и флаги потом

Он мне не понравился

а вот это как раз не важно

это вкусовщина

это вкусовщина

Ну это можно вернуть к старому варианту, не критично, но спасибо за замечание

и вот это https://github.com/alexsadliak/gpm-stats/blob/master/main.go#L119 не надо ли сделать с помощью https://golang.org/pkg/go/build/#hdr-Build_Constraints я не уверен, но сам бы сделал с их помошью, просто потому, что могу :)

google play music desktop версии же нет, но я видел какой-то грязный хак на электроне, для него ты пишешь тулзу?

google play music desktop версии же нет, но я видел какой-то грязный хак на электроне, для него ты пишешь тулзу?

Я там оставил ссылку на этот плеер

Это адаптация веб-версии на електроне

ну значит я был прав, про него я и говорил

Я написал тулзу суто для i3

Чтобы видеть музыку, которая играет

В статусбаре

крутяк, я как раз на i3 сижу

и google music уже 3 год слушаю

Ну такие скрипты были и до меня

Я сделал адаптацию скрипта на пайтоне + баше

Видел два варианта, я решил навелосипедить своё)

и вот это https://github.com/alexsadliak/gpm-stats/blob/master/main.go#L119 не надо ли сделать с помощью https://golang.org/pkg/go/build/#hdr-Build_Constraints я не уверен, но сам бы сделал с их помошью, просто потому, что могу :)

Что именно этими build constraints предлагается сделать? Чтобы невозможно было сбилдить бинарник из других ОС, кроме win, linux, darwin?

с одной строны - это

с другой - правильно инициализировать константу под соответствующую платформу

с другой - правильно инициализировать константу под соответствующую платформу

Ну я о таком подходе не знал. Но насчёт инициализации константы не знаю, потому что есть возможность передать путь к файлу флагом

слу, раз @onokonem закатил такую пьянку - Нил, а глянь и моё поделие, может, найдёшь чего https://github.com/gramework/gramework

Хах)

Масштабы нарастают :)

Как вы сделали плашку code coverage? Не могу её победить в GitLab :)

Стандартный --cover показывает по пакетам только

там же видно в коде страницы

Как вы сделали плашку code coverage? Не могу её победить в GitLab :)

codecov.io, там после настройки в settings -> badge будет код

ну и собственно coverage извлекает travis

А без внешних сервисов есть варианты? У нас корпоративный гитлаб на своём сервере

А без внешних сервисов есть варианты? У нас корпоративный гитлаб на своём сервере

напишите корпоративный бэйдж-генератор

@onokonem глянешь?

ага

@onokonem я вроде все сделал) лог линтера в отдельный файл положил

ну так это, надо пофиксить же!

особенно первые две

ну и результат go fmt запушить

как и результат imports

fmt & imports уже сделаны

как и результат imports

Вроде как он уже

а я вижу один коммит

репа была удалена? а зачем?

не надо так

я перезалил проект :) добавил gitignore... понял усвою

я перезалил проект :) добавил gitignore... понял усвою

люди могут рассчитывать на определенный коммит

ага, усвоил

@onokonem смотрел грэйм?

не успел еще

там немного, если что) cloc . 90 text files. 84 unique files. 14 files ignored. github.com/AlDanial/cloc v 1.70 T=1.14 s (67.5 files/s, 7050.0 lines/s) ------------------------------------------------------------------------------- Language files blank comment code ------------------------------------------------------------------------------- Go 71 964 1214 5451 Markdown 3 119 0 237 YAML 1 7 1 22 Bourne Shell 1 6 2 21 Assembly 1 0 1 0 ------------------------------------------------------------------------------- SUM: 77 1096 1218 5731 -------------------------------------------------------------------------------

я перезалил проект :) добавил gitignore... понял усвою

я там сделал пулреквест и ревью сильно глубоко я не смотрел, прошелся по верхам еще, очевидно, нужен вендоринг

и, возможно, файл с функцией main имеет смысл называть main.go

ну и результат go fmt запушить

кстати для чего go fmt и go imports делать ?

и, возможно, файл с функцией main имеет смысл называть main.go

однозначно, плюсую

@onokonem спасибо !!

@onokonem в остальном с точки зрения обработки коннекта, все ок ? я дальше думаю работать с esme через канал и в канал отправлять ответ, который буду передавать клиенту

@onokonem заметил пару неточностей в PR

@kirilldanshin и тебе спасибо!

да мне-то не за что

кстати для чего go fmt и go imports делать ?

Чтобы код был отформатирован по гайдлайнам го

Чтобы код был отформатирован по гайдлайнам го

если не ошибаюсь goglang сразу это делает

если не ошибаюсь goglang сразу это делает

это даёт всё что угодно

@onokonem https://github.com/therealgofman/smsd только не ругайтесь, я не совсем программист :)

а linter.log это лог чего ?