Added Github action to reproduce races and fixed

go-enry / go-license-detector

Reliable project licenses detector.

Other

127 stars 36 forks source link

Closed g4s8 closed 3 years ago

g4s8 commented 3 years ago

For #9 - reproducing data races in licensedb.Detect(): added workflow for tests with -race flag enabled.

Fixes #9 by wrapping problematic calls with sync.Mutex locks (it's prose chunks parser and rst markdown processor).

g4s8 commented 3 years ago

@bzz could you please check this PR and #9 issue? (lint and windows tests were failing before this PR).

the first commit 544b75f reproduced the problem by enabling built-in data race detector
the second commit b748bcc fixed the problem by wrapping non thread-safe code with mutexes

bzz commented 3 years ago

Thank you for a fix and including a reproducible example in CI, @g4s8