Skip to content

Conversation

@cte
Copy link
Collaborator

@cte cte commented May 22, 2025

Description

pnpm lint wasn't linting all files due to the globbing behavior. I fixed that and then addressed the remaining linter errors.


Important

Fixes linter errors by updating linting configurations and addressing code issues across multiple files.

  • Linting Configuration:
    • Update lint script in package.json files to use eslint with specific extensions and directories.
    • Add .prettierignore to ignore build directories and lock files.
    • Disable specific ESLint rules in eslint.config.mjs and webview-ui/eslint.config.mjs.
  • Code Changes:
    • Replace var with let for variable declarations in index.ts and ClineProvider.ts.
    • Add @ts-expect-error comments for intentional TypeScript errors in modes.test.ts and subtasks.test.ts.
    • Add eslint-disable comments for specific rules in esbuild.ts, git.ts, and TerminalProcess.ts.
    • Fix potential issues with unhandled promise rejections in esbuild.ts and git.ts.
    • Adjust switch case blocks to use braces consistently in anthropic.ts, Task.ts, and export-markdown.ts.
    • Handle optional chaining and nullish coalescing in bedrock.ts and lmstudio.ts.
    • Correctly handle process imports in esbuild.mjs and jest.config.mjs.
    • Fix JSX property warnings in command.tsx and TaskHeader.tsx.
    • Ensure consistent use of const and let in context-mentions.ts and CopyButton.tsx.

This description was created by Ellipsis for 7fccf15. You can customize this summary. It will automatically update as commits are pushed.

@cte cte merged commit 39cd50b into main May 22, 2025
12 checks passed
@cte cte deleted the cte/fix-linter-errors branch May 22, 2025 06:16
@github-project-automation github-project-automation bot moved this from New to Done in Roo Code Roadmap May 22, 2025
@github-project-automation github-project-automation bot moved this from New to Done in Roo Code Roadmap May 22, 2025
hannesrudolph pushed a commit that referenced this pull request May 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

2 participants