Used named constants instead of magic numbers in illumos-specific code

gotestyourself / gotestsum

'go test' runner with output optimized for humans, JUnit XML for CI integration, and a summary of the test results.

Apache License 2.0

1.99k stars 118 forks source link

Used named constants instead of magic numbers in illumos-specific code #375

Closed szaydel closed 9 months ago

szaydel commented 9 months ago

This is based on feedback from @smoynes. I validated that these constants are correctly selected with GOOS=illumos this makes the code more consistent with other already existing OS-specific code in the filewatcher package.

szaydel commented 9 months ago

Thank you for the review and feedback. Cheers!

szaydel commented 9 months ago

Thank you @dnephin! Cheers.