Allan Zhang

Will River have release branches, like current/LTS or nginx mainline/stable? That may influence the MSRV policy.

That makes sense. It was because I saw `UriLauncher::new("myurl")` and `UriLauncher::builder().uri("myurl").build()`, so I wasn't sure whether mandatory parameters should be in the `builder` call or not. Could I still implement...

I tested WIPEONFORK here --> https://github.com/allan2/wipeonfork-pre-exec-example Memory is zeroed both in the child process and before exec when using `pre_exec`. This should cover the typical use case. However, there is...

The default path is ".env" in the current dir. Try specifying the path.

You are probably using the published version, which is 0.15.7 or earlier. You can specify filename in that version using `dotenvy -f env-file`. The filename is searched for in the...

I must've broken the quoted command with my recent changes. I'll add optional file support and fix that soon.

Thanks for this detailed PR. Tasks for review: - See if dotenv libraries in other languages, such as Ruby and go, support this functionality. - POSIX-shell compliance is not a...

@nthuemmel-scontain, thank you for the detailed reply. I'm going to sit on this for now, update the API, and then come back to this potential feature. #19 related

Please read the `std::env::set_var` [safety notice](https://doc.rust-lang.org/std/env/fn.set_var.html#safety). > This makes it not practically possible for you to guarantee that no other thread will read the environment, so the only safe option...

Hi @cprussin, I wouldn't accept a PR but only because it is already planned. Expect a [`load_opt`](https://github.com/allan2/dotenvy/discussions/39#discussioncomment-8833405) function with a `Result` signature. It will return `Error` if there was an...