Skip to content

Refactor(environment): Replace dotenv with node's built in functionality - #139

Merged
jumpy-cat merged 4 commits into
devfrom
replace-dotenv
Nov 14, 2026
Merged

jumpy-cat merged 4 commits into
devfrom
replace-dotenv

Conversation

@amoghvm1

@amoghvm1 amoghvm1 commented Nov 10, 2026 •

Copy link
Copy Markdown
Contributor

Description

Removed dotenv from project entirely in favor of node's built in functionality so that dotenv and the --env-file=.env flag are not both populating variables.

Type of Change

  • New feature (feat)
  • Bug fix (fix)
  • Refactor / code improvement
  • Dependency / build update
  • Documentation
  • Other (explain)

Changed Files

  • package.json
    • Removed dotenv from dependencies
  • mbus.ts
    • Removed dotenv import and dotenv config
  • reminder.ts
    • Removed dotenv import and dotenv config
  • ride.ts
    • Removed dotenv import and dotenv config
  • reminder.test.ts
    • Removed dotenv import and dotenv config

Related Issues

N/A

Does your code rely on another PR

N/A

Testing Done

Flutter:

  • Tested on:
    • iOS Simulator
    • Android Emulator
    • Physical device
      Vitest:
  • existing test(s) with coverage: (which tests?)
  • new tests added: (describe)

Screenshots / Demo (if UI or notification change)

N/A

Checklist

  • Commit messages follow Conventional Commits
  • PR title follows <type>(scope): short description
  • PR target branch is not main and is our current working update branch (e.g. maizebus2.1, dev)
  • No print() / debugPrint() / console.log() left in production code
  • Secrets / keys not committed

@amoghvm1 amoghvm1 linked an issue Nov 10, 2026 that may be closed by this pull request
… actually aren't needed. flags will autopopulate everything before program runs.
… actually aren't needed. flags will autopopulate everything before program runs.
@jumpy-cat
jumpy-cat merged commit 35bb719 into dev Nov 14, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

replace dotenv with node's built-in functionality

3 participants