fix: lint issues #363
fix: lint issues #363
Conversation
|
How to use the Graphite Merge QueueAdd either label to this PR to merge it via the merge queue:
You must have a Graphite account in order to use the merge queue. Sign up using this link. An organization admin has enabled the Graphite Merge Queue in this repository. Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue. This stack of pull requests is managed by Graphite. Learn more about stacking. |
|
| Command | Status | Duration | Result |
|---|---|---|---|
nx affected -t lint typecheck test --parallel -... |
❌ Failed | 6m 49s | View ↗ |
nx affected -t test:e2e --parallel --base=5635d... |
✅ Succeeded | 5m 36s | View ↗ |
☁️ Nx Cloud last updated this comment at 2025-11-12 23:09:42 UTC
jumski
force-pushed
the
11-11-fix-lint-issues
branch
from
9513006 to
313323d
Compare
jumski
force-pushed
the
11-11-fix-lint-issues
branch
from
313323d to
c7f3b20
Compare
![graphite-app[bot]](https://avatars.githubusercontent.com/in/158384?s=60&v=4){kind=small}
| "moduleResolution": "bundler", | ||
| "composite": true |
There was a problem hiding this comment.
Adding 'composite: true' requires additional configuration. Either remove this property or add required settings like 'rootDir' and 'outDir'. Composite projects have stricter requirements that may be causing TypeScript errors.
Spotted by Graphite Agent (based on CI logs)
Is this helpful? React 👍 or 👎 to let us know.
| tsconfigPath: resolve(__dirname, 'tsconfig.lib.json'), | ||
| skipDiagnostics: true // Skip TypeScript diagnostics to avoid vite-plugin-dts errors with monorepo project references. | ||
| // The plugin tries to compile all imported files (including from other packages) | ||
| // which breaks rootDir boundaries. Nx runs proper type checking separately. | ||
| }) | ||
| }) as any // Type cast to avoid Vite version mismatch between packages |
There was a problem hiding this comment.
The property was renamed from 'tsConfigFilePath' to 'tsconfigPath' and a type cast was added. This might be causing compatibility issues. Verify the correct property name for the version of vite-plugin-dts being used and remove the type cast if possible.
Spotted by Graphite Agent (based on CI logs)
Is this helpful? React 👍 or 👎 to let us know.
| // ExtractFlowContext should include FlowContext & custom resources | ||
| type FlowCtx = ExtractFlowContext<typeof flow>; | ||
| // eslint-disable-next-line @typescript-eslint/no-explicit-any | ||
There was a problem hiding this comment.
An eslint-disable comment was removed. If this was intentional, ensure the code doesn't trigger the eslint rule that was being disabled. Otherwise, restore the comment.
Spotted by Graphite Agent (based on CI logs)
Is this helpful? React 👍 or 👎 to let us know.
No description provided.