NAS-142088 / 27.0.0-BETA.1 / Update Containers UI for middleware container.* API changes (container.delete is now a job with force/recursive options) #3090
AlexKarpov98claude.yml
on: pull_request
review
/
...
/
Check member access
4s
review
/
Automatic PR review
11m 34s
Annotations
2 warnings and 3 notices
|
review / check-member / Check member access
Node.js 20 is deprecated. The following actions target Node.js 20 but are being forced to run on Node.js 24: actions/github-script@v7. For more information see: https://github.blog/changelog/2025-09-19-deprecation-of-node-20-on-github-actions-runners/
|
|
review / Automatic PR review
Node.js 20 is deprecated. The following actions target Node.js 20 but are being forced to run on Node.js 24: actions/checkout@v4. For more information see: https://github.blog/changelog/2025-09-19-deprecation-of-node-20-on-github-actions-runners/
|
|
review / Automatic PR review:
src/app/services/network.service.ts#L20
LOW: macRegex is now a bare alias for macAddressRegex with a single consumer that already imports from app/constants/mac-address.constant, leaving two names for one pattern
|
|
review / Automatic PR review:
src/app/pages/containers/components/common/delete-container-dialog/delete-container-dialog.component.html#L17
LOW: The force checkbox is shown and enabled for an already-stopped container where it is inert, and its forceTooltip branch describes running/suspended containers the user is not looking at
|
|
review / Automatic PR review:
src/app/pages/containers/components/all-containers/container-details/container-filesystem-devices/container-filesystem-devices.component.ts#L59
LOW: Disabled Add card action carries no tooltip, unlike the GPU/USB/NIC Add buttons and the per-device actions menu, so the user is not told to stop the container first
|