Yicong-Huang opened a new pull request, #5985: URL: https://github.com/apache/texera/pull/5985
### What changes were proposed in this PR? Fix the two remaining Angular extended-diagnostic warnings surfaced by the frontend build: - **NG8113** — `CardItemComponent` listed `NzWaveDirective` in its standalone `imports` array but never used it in the template. Removed the unused directive import (same class of cleanup as #5975). - **NG8107** — `computing-unit-selection.component.html` used `pve.name?.trim()`, but `PveDraft.name` is typed as a non-nullable `string`, so the optional-chaining `?.` is redundant. Changed to `pve.name.trim()`. No behavior change. ### Any related issues, documentation, discussions? Closes #5984 ### How was this PR tested? Static change only. `PveDraft.name` is `name: string` (non-nullable), so dropping `?.` is type-safe; `NzWaveDirective`'s selector is not referenced in the card-item template. Covered by existing build/CI. ### Was this PR authored or co-authored using generative AI tooling? Generated-by: Claude Code (Claude Opus 4.8) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
