Config/application.php modification suggestions

bedrock
1

Hi there,

I have a little suggest to modification application.php file:
Remove

Dotenv::required(['DB_NAME', 'DB_USER', 'DB_PASSWORD', 'WP_HOME', 'WP_SITEURL']);

or move after

    if (file_exists($env_config)) {
  require_once $env_config;
}

right now definition of constants
DB_NAME
DB_USER
DB_PASSWORD
DB_HOST
in environments file have no effects

This is useful when you need different configuration for DB connection for production or developments WP_ENV
Regards

2

I make this changes in https://github.com/padalec/bedrock/blob/bedrock-dev/config/application.php