core/go

generated from lthn/LEM

fix: CodeRabbit review findings for Env/Path #22

Merged

Virgil merged 1 commit from dev into main

2026-03-22 10:13:16 +00:00

Virgil commented

2026-03-22 10:13:08 +00:00

Member

Fixes 5 CodeRabbit findings: CORE_HOME test override, platform-aware path tests, empty DIR_HOME fallback, doc accuracy.

Fixes 5 CodeRabbit findings: CORE_HOME test override, platform-aware path tests, empty DIR_HOME fallback, doc accuracy.

Virgil added 1 commit 2026-03-22 10:13:08 +00:00

fix: address CodeRabbit review findings 2312801d43

- TestEnv_DIR_HOME checks CORE_HOME override first
- Path tests use Env("DS") instead of hardcoded "/"
- Path() falls back to "." when DIR_HOME is empty
- Doc comment no longer claims "zero filepath import"

Co-Authored-By: Virgil <virgil@lethean.io>

Virgil merged commit fb04b28419 into main

2026-03-22 10:13:16 +00:00

Virgil referenced this pull request from a commit

2026-03-22 10:13:16 +00:00

Merge pull request 'fix: CodeRabbit review findings for Env/Path' (#22) from dev into main

Sign in to join this conversation.

No reviewers

No labels

No milestone

No project

No assignees

1 participant

Notifications

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: core/go#22

No description provided.