feat(core): specify a custom .env file for workspace:run-commands

[rarmatei]

Expected Behavior (This is the new behavior we can expect after the PR is merged)

By default @nrwl/workspace:run-commands looks for a .env file at the root level, and loads all the variables in there before running the command.

Sometimes, it might be useful to specify different dotenv depending on what environment you're targetting.

With this PR, you can specify a envFile option to run-commands which loads that instead of the root .env:

       "sentry-push-sourcemaps-prod": {
          "builder": "@nrwl/workspace:run-commands",
          "options": {
            "commands": [
              {
                "envFile": "apps/my-app/environments/prod.env",
                "command": "nx run my-app --target=sentry-push-sourcemaps"
              }
            ]
          }
        },

Issue

Closes #2199

[rarmatei]

since it's a "hidden" default we don't throw if the user doesn't have a root .env file
we do throw however, if they specify explicitly and it doesn't exist

[jaysoo]

It makes sense. 👍

[nelsieborja]

Would it be possible to add the same .env customization for next builders?

nx/packages/next/src/builders/dev-server/dev-server.impl.ts

Lines 20 to 22 in 9c50569

	try {
	require('dotenv').config();
	} catch (e) {}

Some of our old codes are using process.env and we cannot change them to use:
import { environment } from '../environments/environment';

[asciidiego]

In case somebody wonders why the code from the original question does not work, it's because the envFile option should go, well... inside the options key. See example below:

       "sentry-push-sourcemaps-prod": {
          "builder": "@nrwl/workspace:run-commands",
          "options": {
            "envFile": "apps/my-app/environments/prod.env", // This SHOULD go here
            "commands": [
              {
                "envFile": "apps/my-app/environments/prod.env", // NOT here
                "command": "nx run my-app --target=sentry-push-sourcemaps"
              }
            ]
          }
        },

Took me 20 minutes and some source code browsing to figure it out.

[github-actions]

This pull request has already been merged/closed. If you experience issues related to these changes, please open a new issue referencing this pull request.