fix(js): libraries should be buildable by default

[mandarini]

Current Behavior

Recent updates to the @nrwl/js:lib generator changed the previous default behaviour. Before version 15.8.5, the @nrwl/js:lib generator would, by default, generate a buildable lib. In the schema.json buildable had a default value of true, and it would default to @nrwl/js:tsc as the build executor. The only way to get a non-buildable library would be to set buildable=false specifically.

This behaviour changed in the deprecation of the buildable option, and the use of bundler, since bundler would be set to none by default if not provided.

Expected Behavior

The new generator should respect the old functionality of having buildable libraries by default, and a library should not be buildable only if bundler is specifically set to none.

Related Issue(s)

Fixes #15516

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated
nx-dev	✅ Ready (Inspect)	Visit Preview	💬 Add your feedback	Mar 8, 2023 at 3:28PM (UTC)

[nx-cloud]

☁️ Nx Cloud Report

CI is running/has finished running commands for commit 3e1f6a2. 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 8 targets

• nx run-many -t e2e-macos --parallel=1 --exclude=e2e-detox
• nx affected --target=e2e --base=5d19b53abc35e2f84d7d27ca560ec4c1a80c5665 --head=3e1f6a21926f50faf94373920baf2e297fafa199 --exclude=e2e-storybook,e2e-storybook-angular --parallel=1
• nx affected --target=test --base=5d19b53abc35e2f84d7d27ca560ec4c1a80c5665 --head=3e1f6a21926f50faf94373920baf2e297fafa199 --parallel=1
• nx affected --target=build --base=5d19b53abc35e2f84d7d27ca560ec4c1a80c5665 --head=3e1f6a21926f50faf94373920baf2e297fafa199 --parallel=3
• nx run-many -t root-lint check-imports check-commit check-lock-files depcheck --parallel=1 --no-dte
• nx affected --target=lint --base=5d19b53abc35e2f84d7d27ca560ec4c1a80c5665 --head=3e1f6a21926f50faf94373920baf2e297fafa199 --parallel=3
• nx-cloud record -- nx format:check --base=5d19b53abc35e2f84d7d27ca560ec4c1a80c5665 --head=3e1f6a21926f50faf94373920baf2e297fafa199
• nx documentation --no-dte

---

Sent with 💌 from NxCloud.

[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.