Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

docs(nx-dev): article on task inference #29330

Merged
merged 1 commit into from
Dec 18, 2024

Conversation

juristr
Copy link
Member

@juristr juristr commented Dec 12, 2024

Current Behavior

Expected Behavior

Related Issue(s)

Fixes #

Copy link

vercel bot commented Dec 12, 2024

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

Name Status Preview Updated (UTC)
nx-dev ✅ Ready (Inspect) Visit Preview Dec 18, 2024 4:09pm

Copy link

nx-cloud bot commented Dec 12, 2024

Your CI Pipeline Execution ↗ for commit 83e2a7f ⏳ is in progress.

Command Status Duration Result
nx affected --targets=lint,test,build,e2e,e2e-c... ✅ Succeeded 4m, 40s View ↗
nx run-many -t check-imports check-commit check... ✅ Succeeded 59s View ↗
nx-cloud record -- nx-cloud conformance:check ✅ Succeeded 0s View ↗
nx-cloud record -- nx format:check --base=5bdda... ✅ Succeeded 21s View ↗
nx-cloud record -- nx sync:check ✅ Succeeded 20s View ↗
nx documentation --no-dte ✅ Succeeded 44s View ↗

Last updated at 2024-12-13 14:47:08UTC

Sent with 💌 from NxCloud.

Copy link

nx-cloud bot commented Dec 18, 2024

Your CI Pipeline Execution ↗ for commit 27474b7 is in progress ⏳

Command Status Duration Result
nx affected --targets=lint,test,build,e2e,e2e-c... ✅ Succeeded 3m 5s View ↗
nx run-many -t check-imports check-commit check... ✅ Succeeded 1m 9s View ↗
nx-cloud record -- nx-cloud conformance:check ✅ Succeeded <1s View ↗
nx-cloud record -- nx format:check --base=6051e... ✅ Succeeded 26s View ↗
nx-cloud record -- nx sync:check ✅ Succeeded 25s View ↗
nx documentation --no-dte ✅ Succeeded 45s View ↗

☁️ Nx Cloud last updated this comment at 2024-12-18 16:06:20 UTC

Copy link
Collaborator

@isaacplmann isaacplmann left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Wording changes and added a note about having a tsconfig that includes your plugin file

docs/blog/2024-12-18-dynamic-targets-with-inference.md Outdated Show resolved Hide resolved
docs/blog/2024-12-18-dynamic-targets-with-inference.md Outdated Show resolved Hide resolved
docs/blog/2024-12-18-dynamic-targets-with-inference.md Outdated Show resolved Hide resolved
To see whether you plugin loaded properly you can go to `.nx/workspace-data/d/daemon.log` and search for your plugin name. Behind the scenes the [Nx Daemon](/concepts/nx-daemon) re-calculates the project graph and loads all plugins, including ours.

{% callout type="info" title="TypeScript configuration" %}
Make sure you have some `tsconfig.json` file in the monorepo root. Nx loads the plugin dynamically (without you having to precompile it) which requires some TypeScript context to be present.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
Make sure you have some `tsconfig.json` file in the monorepo root. Nx loads the plugin dynamically (without you having to precompile it) which requires some TypeScript context to be present.
Make sure you have some `tsconfig.json` file in the monorepo root and a TS config file that includes the `tools/storybook.ts` file. Nx loads the plugin dynamically (without you having to precompile it) which requires some TypeScript context to be present.

@juristr juristr force-pushed the article/inference-tasks branch from 533f72e to 27474b7 Compare December 18, 2024 15:56
@juristr juristr enabled auto-merge (squash) December 18, 2024 15:58
@juristr juristr merged commit 7ab5945 into nrwl:master Dec 18, 2024
5 of 6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants