search

amit-davidson / Chronos

Chronos - A static race detector for the go language
MIT License
424 stars 11 forks source link

Minor fixes #10

Closed ninedraft closed 3 years ago

ninedraft commented 3 years ago

Minor style and performance fixes

amit-davidson commented 3 years ago

Wow, thank you very much for your effort. I'll look at all the PRs in the evening.

cristaloleg commented 3 years ago

Looks like this PR contains few other commits from other PRs.

ninedraft commented 3 years ago

Looks like this PR contains few other commits from other PRs.

Shame on me

Fixed

amit-davidson commented 3 years ago

Also, you can run tests locally. They are located under e2e-tests/summary_test.go.

ninedraft commented 3 years ago

Fixed remarks

amit-davidson commented 3 years ago

@ninedraft You can click the re-review button if it's ready