1) WorkerPool дублирует название пакета. Лучше назвать Pool, тогда вызов его будет worker.Pool, а не worker.WorkerPool.
2) WorkersCount - экспортируемая константа, но снаружи не используется. Лучше изначально константы делать неэкспортируемыми - workersCount
И WorkersCount можно передавать снаружи настройкой.
andrey-priemko
commented
5 years ago
1) WorkerPool дублирует название пакета. Лучше назвать Pool, тогда вызов его будет worker.Pool, а не worker.WorkerPool.
2) WorkersCount - экспортируемая константа, но снаружи не используется. Лучше изначально константы делать неэкспортируемыми - workersCount И WorkersCount можно передавать снаружи настройкой.
3) В пакете github.com/andrey-priemko/go_study/thumbgenerator/model в структуре Task поле Id -> ID. Url аналогично. См. https://github.com/golang/go/wiki/CodeReviewComments#initialisms