Remove post install .env copy

[swalkinshaw]

.env.example is just an example and doesn't contain useful defaults. Copying this file can be confusing since it needs to be edited regardless. By not copying this file, we'll surface errors of missing config values sooner since they'll be no default values.

This is an alternative to #587 and #586

Docs PR: roots/docs#476

[aaemnnosttv]

I find the .env.example to be useful as a guide (like the docs currently say) as to what values are needed and have actively added to it in the past as a reference for new devs.

Also, this change would break my wp-cli valet command which replaces these defaults when creating a new Bedrock site. I'm not saying this change shouldn't happen for that reason only, but it would be nice to have some time to prepare an update before this is merged, if it goes through.

Edit: I may have misunderstood the change here – I read this as .env.example was being removed, but now I understand this is only about not copying it automatically after the initial install (as the title says 🤦‍♂️ ), in which case this should be fine 😄

[Jamiewarb]

I like this change. I think it's great for new players to Bedrock that might not be familiar with it. Following a README that asks to explicitly copy .env.example to .env will make them away of this file from the get-go.

[tangrufus]

Suggestion: if .env if not exists, die() with an error messge and link to roots.io/docs before

bedrock/config/application.php

Line 32 in e940020

$dotenv = Dotenv\Dotenv::createUnsafeImmutable($root_dir, ['.env', '.env.local'], false);

Similar to https://github.com/roots/sage/blob/6d66debe17dc413cd9f276ddb42a368bf0dc844d/functions.php#L15

[austinpray]

Suggestion: if .env if not exists, die() with an error messge and link to roots.io/docs before

bedrock/config/application.php

Line 32 in e940020

$dotenv = Dotenv\Dotenv::createUnsafeImmutable($root_dir, ['.env', '.env.local'], false);

Similar to https://github.com/roots/sage/blob/6d66debe17dc413cd9f276ddb42a368bf0dc844d/functions.php#L15

This doesn't work for dockerized environments where the environment variables are set by the container orchestrator (Kubernetes, heroku, etc). .env should only be used if you don't have first-class environment variable management.

[ggiak]

@swalkinshaw
check #674, should do the trick without hussle :)