Skip to content

chore: use rslint in rslint#41

Merged
hardfist merged 6 commits intomainfrom
yj-use-lint
Jul 23, 2025
Merged

chore: use rslint in rslint#41
hardfist merged 6 commits intomainfrom
yj-use-lint

Conversation

@hardfist
Copy link
Copy Markdown
Contributor

No description provided.

@hardfist hardfist requested a review from Copilot July 23, 2025 02:52
@hardfist
Copy link
Copy Markdown
Contributor Author

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR configures the rslint project to use its own linting tool (rslint) for code quality checks. The changes establish a consistent linting workflow across the project and integrate it into the CI pipeline.

  • Adds lint scripts to package.json files to enable running rslint across the project
  • Updates TypeScript configuration and binary settings for the rslint package
  • Integrates linting into the GitHub Actions CI workflow
  • Removes redundant Go module caching and simplifies CI configuration

Reviewed Changes

Copilot reviewed 6 out of 7 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
packages/vscode-extension/package.json Adds lint script using rslint with TypeScript configuration
packages/vscode-extension/tsconfig.build.json Updates extends path to use root-level TypeScript build configuration
packages/rslint/package.json Updates binary name and adds verbose flag to Go build
package.json Adds root-level lint script to run linting across all packages
.prettierignore Excludes additional VS Code test directory from formatting
.github/workflows/ci.yml Integrates linting step and simplifies Go setup configuration

@hardfist hardfist merged commit a8f95ec into main Jul 23, 2025
3 checks passed
@hardfist hardfist deleted the yj-use-lint branch July 23, 2025 02:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants