feat: support container detection
Resolves #140
@pi0 is it okay? I also added the wsl and other containers' checks to cover eventually future requests.
Codecov Report
Attention: Patch coverage is 0% with 76 lines in your changes missing coverage. Please review.
Project coverage is 52.63%. Comparing base (
5558f03) to head (a1ff14b). Report is 6 commits behind head on main.
Additional details and impacted files
@@ Coverage Diff @@
## main #141 +/- ##
===========================================
- Coverage 82.08% 52.63% -29.46%
===========================================
Files 10 14 +4
Lines 374 285 -89
Branches 33 35 +2
===========================================
- Hits 307 150 -157
- Misses 61 125 +64
- Partials 6 10 +4
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Thanks. Since wsl is not exactly a container (it is VM with dedicated kernel), i would suggest to use /ws and /docker subpaths.
I would love to explore idea of detecting other container runtimes (from cgroups perhaps?) we can iterate later.
You're right, I just didn't want to make another folder just for that ahah Anyway okay I'll separate them, should I remove the other container runtimes' checks? I'll try to add tests for these new flags, hoping not to mess it up ahah
(sorry for late review on this -- it is definitely under my radar)