Skip to content

chore: add check meta-task for org consistency#10

Merged
nsheaps merged 1 commit intomainfrom
standardize-mise-tasks
Feb 24, 2026
Merged

chore: add check meta-task for org consistency#10
nsheaps merged 1 commit intomainfrom
standardize-mise-tasks

Conversation

@nsheaps
Copy link
Owner

@nsheaps nsheaps commented Feb 23, 2026

Summary

  • Adds a check meta-task to mise.toml that runs both lint and test
  • Part of org-wide mise standardization across nsheaps repos

Test plan

  • Run mise run check and verify it executes both lint and test
  • Verify mise run lint and mise run test still work independently

🤖 Generated with Claude Code

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Copy link
Owner Author

@nsheaps nsheaps left a comment

Choose a reason for hiding this comment

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

Review: chore: add check meta-task for org consistency — Score: 97/100

Category Score Notes
Simplicity 100 Single task definition, clear structure
Flexibility 100 Depends on existing lint and test tasks
Usability 100 Meta-task pattern aligns with org standard
Documentation 100 Clear description: "Run all checks (lint + test)"
Security 100 No security implications
Pattern Matching 100 Matches git-wt repo mise.toml pattern
Best Practices 100 Proper depends syntax, echo feedback message included
General QA 97 Lint and test tasks verified to exist; check task correctly specified

✅ All categories ≥85% — Ready to merge

Verdict: Clean check meta-task addition. Depends chain correct. Approve.

@nsheaps nsheaps added the ready All review categories 85%+ — ready to merge label Feb 24, 2026
@nsheaps nsheaps marked this pull request as ready for review February 24, 2026 02:45
@nsheaps nsheaps merged commit cfc08a4 into main Feb 24, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready All review categories 85%+ — ready to merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant