Skip to content

Conversation

podtserkovskiy
Copy link
Member

No description provided.

Mikhail Podtserkovskiy added 2 commits June 14, 2017 16:30
@@ -0,0 +1,191 @@
package main
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Поставь ищью на рефакторинг, а пока вставь коммент, что прототип. ВЫглядит ужасно

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ок, так это и есть по факту "первый блин"

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Просто расскажи всем об этом, пока только мы знаем 😉

test:
go test ./...

concurrency-test-prepare: build
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Хмммм, сомнительно нужно ли это здесь. Я думаю что это все таки проблемы самого разработчика ставить что-то к себе, а самое главное что у тебя и обновлять будет автоматически, ты точно этого хочешь?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ему так или иначе придется что то поставить чтобы это что то запустить.
Как предлагаешь решить эту проблему? go run ...?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ну мне кажется что либо вендор, либо просто указание зависимостей.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Понял зачем все это, напиши что ты делаешь, что бы пользователь не переживал что у него кто- там что-то инсталит.

@andrskom andrskom merged commit ae2c5e7 into master Jun 15, 2017
@podtserkovskiy podtserkovskiy deleted the concurrency-tests branch June 20, 2017 19:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants