Skip to content

ci: enable duplication linter in CI#535

Merged
yinwm merged 2 commits intosipeed:mainfrom
xiaket:ci-enable-dupl-linter
Mar 2, 2026
Merged

ci: enable duplication linter in CI#535
yinwm merged 2 commits intosipeed:mainfrom
xiaket:ci-enable-dupl-linter

Conversation

@xiaket
Copy link
Collaborator

@xiaket xiaket commented Feb 20, 2026

📝 Description

Enable duplication linter in CI so as to defend against AI slop.

And some refactors to remove the duplications.

🗣️ Type of Change

  • 🐞 Bug fix (non-breaking change which fixes an issue)
  • ✨ New feature (non-breaking change which adds functionality)
  • 📖 Documentation update
  • ⚡ Code refactoring (no functional changes, no api changes)

🤖 AI Code Generation

  • 🤖 Fully AI-generated (100% AI, 0% Human)
  • 🛠️ Mostly AI-generated (AI draft, Human verified/modified)
  • 👨‍💻 Mostly Human-written (Human lead, AI assisted or none)

☑️ Checklist

  • My code/docs follow the style of this project.
  • I have performed a self-review of my own changes.
  • I have updated the documentation accordingly.

@xiaket xiaket force-pushed the ci-enable-dupl-linter branch 2 times, most recently from a69d9fc to b4bd2e6 Compare February 24, 2026 10:42
@xiaket xiaket requested a review from imguoguo February 24, 2026 10:44
@imguoguo imguoguo requested a review from yinwm February 24, 2026 11:10
@xiaket xiaket force-pushed the ci-enable-dupl-linter branch from b4bd2e6 to f41be58 Compare February 25, 2026 12:36
@yinwm
Copy link
Collaborator

yinwm commented Feb 26, 2026

@xiaket please fix this

@xiaket xiaket force-pushed the ci-enable-dupl-linter branch from e5f5cb7 to efdae09 Compare February 26, 2026 09:16
@xiaket
Copy link
Collaborator Author

xiaket commented Feb 26, 2026

@yinwm rebased, please take a look. 🙇

@xiaket xiaket force-pushed the ci-enable-dupl-linter branch from efdae09 to 6d45bae Compare March 1, 2026 06:59
Signed-off-by: Kai Xia <[email protected]>
@xiaket xiaket force-pushed the ci-enable-dupl-linter branch from 6d45bae to 32c864c Compare March 1, 2026 07:20
Signed-off-by: Kai Xia <[email protected]>
@yinwm
Copy link
Collaborator

yinwm commented Mar 2, 2026

Code review

No issues found. Checked for bugs and CLAUDE.md compliance.

🤖 Generated with Claude Code

- If this code review was useful, please react with 👍. Otherwise, react with 👎.

Copy link
Collaborator

@yinwm yinwm left a comment

Choose a reason for hiding this comment

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

LGTM

@yinwm yinwm merged commit faec026 into sipeed:main Mar 2, 2026
2 checks passed
hyperwd pushed a commit to hyperwd/picoclaw that referenced this pull request Mar 5, 2026
Pluckypan pushed a commit to Pluckypan/picoclaw that referenced this pull request Mar 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants