Skip to content

Feature/navigation#11

Draft
KelianHalleray wants to merge 3 commits into
developfrom
feature/navigation
Draft

Feature/navigation#11
KelianHalleray wants to merge 3 commits into
developfrom
feature/navigation

Conversation

@KelianHalleray

Copy link
Copy Markdown

PR Type

Summary

Description

What's Changed

  • [ ]
  • [ ]
  • [ ]

Screen

Will


@KelianHalleray KelianHalleray self-assigned this Jun 10, 2026
@KelianHalleray KelianHalleray added the feature new functionality label Jun 10, 2026
@qodo-code-review

Copy link
Copy Markdown

CI Feedback 🧐

A test triggered by this PR failed. Here is an AI-generated analysis of the failure:

Action: Lint, format, typecheck, build

Failed stage: Lint [❌]

Failed test name: ""

Failure summary:

The action failed during the ESLint step because lint warnings are treated as failures
(--max-warnings 0).
- ESLint reported 1 warning in src/components/dashboard/camera-feed.tsx at 32:15
from rule @next/next/no-img-element (using instead of Next.js ).
- Since the workflow
allows a maximum of 0 warnings, ESLint exited with code 1 (ESLint found too many warnings (maximum:
0).).

Relevant error logs:
1:  ##[group]Runner Image Provisioner
2:  Hosted Compute Agent
...

231:  + jsdom 25.0.1
232:  + lint-staged 15.5.2
233:  + prettier 3.8.1
234:  + tailwindcss 4.2.1
235:  + typescript 5.9.3
236:  + vitest 2.1.9
237:  Done in 9.9s
238:  ##[group]Run pnpm exec eslint . --max-warnings 0
239:  �[36;1mpnpm exec eslint . --max-warnings 0�[0m
240:  shell: /usr/bin/bash -e {0}
241:  env:
242:  PNPM_HOME: /home/runner/setup-pnpm/node_modules/.bin
243:  ##[endgroup]
244:  /home/runner/work/robot-front/robot-front/src/components/dashboard/camera-feed.tsx
245:  ##[warning]  32:15  warning  Using `<img>` could result in slower LCP and higher bandwidth. Consider using `<Image />` from `next/image` or a custom image loader to automatically optimize images. This may incur additional usage or cost from your provider. See: https://nextjs.org/docs/messages/no-img-element  @next/next/no-img-element
246:  ✖ 1 problem (0 errors, 1 warning)
247:  ESLint found too many warnings (maximum: 0).
248:  ##[error]Process completed with exit code 1.
249:  Post job cleanup.

@damnthonyy damnthonyy left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

increase --max-warnings to 4 like and its good. Its just a warning

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature new functionality

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants