فهرست منبع

Remove eslint-disable, forbid it in .clinerules

Matt Rubens 1 سال پیش
والد
کامیت
d1e305f30c
2فایلهای تغییر یافته به همراه6 افزوده شده و 2 حذف شده
  1. 6 0
      .clinerules
  2. 0 2
      webview-ui/src/components/chat/__tests__/ChatTextArea.test.tsx

+ 6 - 0
.clinerules

@@ -18,6 +18,12 @@
    - Keep the documentation in sync with the codebase
    - Add examples where appropriate
 
+4. Lint Rules:
+   - Never disable any lint rules without explicit user approval
+   - If a lint rule needs to be disabled, ask the user first and explain why
+   - Prefer fixing the underlying issue over disabling the lint rule
+   - Document any approved lint rule disabling with a comment explaining the reason
+
 # Adding a New Setting
 
 To add a new setting that persists its state, follow these steps:

+ 0 - 2
webview-ui/src/components/chat/__tests__/ChatTextArea.test.tsx

@@ -1,5 +1,3 @@
-/* eslint-disable import/first */
-import React from 'react';
 import { render, fireEvent, screen } from '@testing-library/react';
 import '@testing-library/jest-dom';
 import ChatTextArea from '../ChatTextArea';