Skip to content

Add dotenv support for arguments - #327

Open
emma31-dev wants to merge 2 commits into
bitcoindevkit:masterfrom
emma31-dev:feat/load-args-from-env
Open

emma31-dev wants to merge 2 commits into
bitcoindevkit:masterfrom
emma31-dev:feat/load-args-from-env

Conversation

@emma31-dev

@emma31-dev emma31-dev commented Sep 17, 2026

Copy link
Copy Markdown

Description

This pr Introduce an env feature that loads arguments from a .env file via dotenvy, enabled by default, instead of manually typing args in the command as it gets difficult to change values frequently. Overtime the command of a single action can balloon to over 80 chars with a lot of them repeating over different commands. since it already support environment variables , .env support seems like a logical next step IMO.

Changelog notice

  • Add env feature that loads arguments from a .env file via dotenvy

Checklists

All Submissions:

  • I've signed all my commits
  • I followed the contribution guidelines
  • I ran cargo fmt and cargo clippy before committing

New Features:

* [ ] I've added tests for the new feature

  • I've added docs for the new feature
  • I've updated CHANGELOG.md

Introduce an `env` feature that loads arguments from a .env file
via dotenvy, disabled by default.
@emma31-dev
emma31-dev marked this pull request as draft September 17, 2026 12:07
@emma31-dev
emma31-dev force-pushed the feat/load-args-from-env branch 2 times, most recently from 00f2dd0 to 6c89115 Compare September 17, 2026 21:34
@emma31-dev
emma31-dev marked this pull request as ready for review September 17, 2026 21:34
@emma31-dev emma31-dev changed the title Add optional dotenv support for arguments Add dotenv support for arguments Sep 17, 2026
The `env` feature is now included in the default features, removing
the need for the conditional compilation guard around the dotenv call.

Signed-off-by: emma31-dev <emmanuelfidel07@gmail.com>
@emma31-dev
emma31-dev force-pushed the feat/load-args-from-env branch from 6c89115 to 4034a84 Compare September 17, 2026 21:53
@emma31-dev

Copy link
Copy Markdown
Author

@tvpeter This pr is ready for your review

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

1 participant