-
Notifications
You must be signed in to change notification settings - Fork 84
Move env-file loading into a shared helper #255
Copy link
Copy link
Closed
Labels
GrantFox OSSIssue tracked in GrantFox OSSIssue tracked in GrantFox OSSMaybe RewardedIssue may be eligible for a GrantFox rewardIssue may be eligible for a GrantFox rewardOfficial Campaign | FWC26Campaign: Official Campaign | FWC26Campaign: Official Campaign | FWC26area:backendBackend (NestJS) areaBackend (NestJS) areakind:refactorRefactor / cleanupRefactor / cleanup
Description
Activity
Metadata
Metadata
Assignees
Labels
GrantFox OSSIssue tracked in GrantFox OSSIssue tracked in GrantFox OSSMaybe RewardedIssue may be eligible for a GrantFox rewardIssue may be eligible for a GrantFox rewardOfficial Campaign | FWC26Campaign: Official Campaign | FWC26Campaign: Official Campaign | FWC26area:backendBackend (NestJS) areaBackend (NestJS) areakind:refactorRefactor / cleanupRefactor / cleanup
Problem Statement. Env-file resolution appears twice:
main.tsandapp.module.ts. Slight drift is possible between them.Technical Context.
app/backend/src/main.ts,app/backend/src/app.module.ts.Expected Outcome. A
loadEnv()helper exported bysrc/common/utils/env-loader.tsis the only place env files are searched;main.tsandapp.module.tsboth call it.Acceptance Criteria.
candidate env files.
dotenvalways wins over the OS env (or the opposite, with adocumented order).
Files or modules likely to be affected.
new
src/common/utils/env-loader.ts,main.ts,app.module.ts.Difficulty. Easy
Estimated effort. XS
Backlog item #47 from `docs/maintainer-issue-backlog.md.