Skip to content

concurrency tests - #3

Merged
andrskom merged 3 commits into
masterfrom
concurrency-tests
Jun 15, 2017
Merged

andrskom merged 3 commits into
masterfrom
concurrency-tests

Conversation

@podtserkovskiy

Copy link
Copy Markdown
Member

No description provided.

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

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

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

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

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

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

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

Comment thread Makefile
test:
go test ./...

concurrency-test-prepare: build

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

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

Copy link
Copy Markdown
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
Copy Markdown
Member

Choose a reason for hiding this comment

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

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

Copy link
Copy Markdown
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