Angular: Do not use default for includePaths

[Coly010]

Issue: In Nx Workspaces, the CLI evaluates the schema.json file slightly differently to how Angular CLI evaluates it.

The Angular CLI will only look at the parent object's default value and return that when the option is not provided.

The Nx CLI will look at the parent object's childrens' default values and build out an object that matches those defaults, as would be expected.

This leads to erroneous behaviors in Nx Workspaces, as the stylePreprocessorOptions coming from the app's browser build target will get overridden by an object of the form:

{
  stylePreprocessorOptions: {
    includePaths: []
  }
}

What I did

I removed the default empty array form includePaths as this is the source of the erroneous behavior. It also does not need to have a default value as it is an optional property.

How to test

• Is this testable with Jest or Chromatic screenshots?
• Does this need a new example in the kitchen sink apps?
• Does this need an update to the documentation?

If your answer is yes to any of these, please make sure to include it in your PR.

[nx-cloud]

☁️ Nx Cloud Report

CI is running/has finished running commands for commit eadc4fd. As they complete they will appear below. Click to see the status, the terminal output, and the build insights.

📂 See all runs for this branch

---

✅ Successfully ran 1 target

• nx run-many --target=prepare --all --parallel --max-parallel=15 -- --optimized

---

Sent with 💌 from NxCloud.

[mandarini]

Nice!!!! :D :D @shilman do you think this modification makes sense? It's been causing some issues on our side! :)

[kylegach]

Hey, @kroeder — Can you take a look at this, please?

[kroeder]

LGTM, thanks for the PR! 🙂

[Coly010]

@kroeder let me know if there is anything to be done with the failing tests. I'm not sure how they could have failed given I didn't touch the areas of the failing tests? (Vue, CRA?)

[kroeder]

I'm not sure actually. I heard yesterday there are issues with the pipeline

Anything we can do about the failing steps @ndelangen @shilman?
Chromatic fails, but it generates a link and the demo works as expected

[shilman]

@kroeder we're working on those CI failures, which are unrelated to this PR. Merging! Thanks everybody!

[Coly010]

Thanks!!

[mandarini]

Whoohooo!!! :D