Skip to content

DevX: minor improvements in Makefile#726

Merged
yinwm merged 1 commit intosipeed:mainfrom
xiaket:devx-make-improvements
Feb 24, 2026
Merged

DevX: minor improvements in Makefile#726
yinwm merged 1 commit intosipeed:mainfrom
xiaket:devx-make-improvements

Conversation

@xiaket
Copy link
Collaborator

@xiaket xiaket commented Feb 24, 2026

📝 Description

  1. Add a target so we can run make fix to fix issues in code(also added to make check)
  2. Add proper indentation for targets in make help.

🗣️ 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)

🔗 Related Issue

📚 Technical Context (Skip for Docs)

  • **Reference URL:✱
  • **Reasoning:✱

🧪 Test Environment

  • Hardware:
  • OS:
  • Model/Provider:
  • Channels:

📸 Evidence (Optional)

Click to view Logs/Screenshots

☑️ 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.

Signed-off-by: Kai Xia <kaix+github@fastmail.com>
@xiaket xiaket requested a review from imguoguo February 24, 2026 11:07
@xiaket xiaket force-pushed the devx-make-improvements branch from d425967 to 78e5bda Compare February 24, 2026 11:22
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 - author addressed the feedback by removing fix from the check target. Good quality of life improvements.

@yinwm
Copy link
Collaborator

yinwm commented Feb 24, 2026

LGTM

@yinwm yinwm merged commit b10555c into sipeed:main Feb 24, 2026
2 checks passed
hyperwd pushed a commit to hyperwd/picoclaw that referenced this pull request Mar 5, 2026
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