hexlet-rus / runit

Online Editor Form Embedding
https://runit.hexlet.ru/
Other
34 stars 114 forks source link

Ворнинги после запуска приложения #495

Closed fey closed 1 month ago

fey commented 1 month ago

после запуска приложения вываливаются ворнинги

Compiled with warnings.

[eslint] 
src/components/Forms/AvatarChangeForm.jsx
  Line 16:9:  'handleInDevelopment' is assigned a value but never used  @typescript-eslint/no-unused-vars

src/components/Modals/ChangeAvatar.jsx
  Line 8:9:  'dispatch' is assigned a value but never used  @typescript-eslint/no-unused-vars

src/components/Modals/NewSnippet.jsx
  Line 32:7:  'extensions' is assigned a value but never used  @typescript-eslint/no-unused-vars

src/components/Modals/RemoveAvatar.jsx
  Line 6:9:  'dispatch' is assigned a value but never used  @typescript-eslint/no-unused-vars

src/components/Navigation/UserMenu.jsx
  Line 20:9:  'handleInDevelopment' is assigned a value but never used  @typescript-eslint/no-unused-vars

Search for the keywords to learn more about each warning.
To ignore, add // eslint-disable-next-line to the line before.

WARNING in [eslint] 
src/components/Forms/AvatarChangeForm.jsx
  Line 16:9:  'handleInDevelopment' is assigned a value but never used  @typescript-eslint/no-unused-vars

src/components/Modals/ChangeAvatar.jsx
  Line 8:9:  'dispatch' is assigned a value but never used  @typescript-eslint/no-unused-vars

src/components/Modals/NewSnippet.jsx
  Line 32:7:  'extensions' is assigned a value but never used  @typescript-eslint/no-unused-vars

src/components/Modals/RemoveAvatar.jsx
  Line 6:9:  'dispatch' is assigned a value but never used  @typescript-eslint/no-unused-vars

src/components/Navigation/UserMenu.jsx
  Line 20:9:  'handleInDevelopment' is assigned a value but never used  @typescript-eslint/no-unused-vars

webpack compiled with 1 warning
No issues found.
Maiiiiiiiiia commented 1 month ago

возьму в работу

fey commented 1 month ago

Я чинил отправку писем и возможно поправил в этом коммите https://github.com/hexlet-rus/runit/commit/a3a47ab40fe5e69bd1124b1d79003005ed643861 :thinking: но проверить все равно стоит

Maiiiiiiiiia commented 1 month ago

варнинги остались. а подскажите, их убираем комментариями для отключения правила? Или их можно закоммитить?

fey commented 1 month ago

Коммитим-исправляем конечно же) Заглушать замечания линтера - это крайний путь (и часто не самый верный)

HelenOne commented 1 month ago

@Maiiiiiiiiia если беретесь за задачку поставьте себя как assignee тогда

Maiiiiiiiiia commented 1 month ago

а как поменять assignee на меня?

HelenOne commented 1 month ago

Поменяла на вас

fey commented 1 month ago

@Maiiiiiiiiia я смержил. Проверьте. пожалуйста. Если все ок, то закрою.

Maiiiiiiiiia commented 1 month ago

все ок!