Skip to content

refactor: replace Makefile with cargo xtask#159

Merged
sachiniyer merged 1 commit intomainfrom
siyer/xtask
Mar 8, 2026
Merged

refactor: replace Makefile with cargo xtask#159
sachiniyer merged 1 commit intomainfrom
siyer/xtask

Conversation

@sachiniyer
Copy link
Copy Markdown
Contributor

@sachiniyer sachiniyer commented Mar 7, 2026

  • Add xtask/ workspace crate with commands:
    • cargo xtask check — run all vendored-artifact checks
    • cargo xtask check-help / generate-help — HELP.md management
    • cargo xtask check-openapi / generate-openapi — OpenAPI spec management
  • Move openapi.json comparison from curl+python+diff to Rust (serde_json)
  • Move generate_help from examples/ to xtask
  • Add .cargo/config.toml with xtask alias
  • Update CI to use cargo xtask check instead of make check
  • Delete Makefile and examples/generate_help.rs

Co-Authored-By: Claude Opus 4.6 noreply@anthropic.com


Open with Devin

Copy link
Copy Markdown
Contributor Author

This stack of pull requests is managed by Graphite. Learn more about stacking.

@sachiniyer sachiniyer marked this pull request as ready for review March 7, 2026 09:28
@chatgpt-codex-connector
Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

Copy link
Copy Markdown
Contributor

@devin-ai-integration devin-ai-integration Bot left a comment

Choose a reason for hiding this comment

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

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no potential bugs to report.

View in Devin Review to see 3 additional findings.

Open in Devin Review

- Add `xtask/` workspace crate with commands:
  - `cargo xtask check` — run all vendored-artifact checks
  - `cargo xtask check-help` / `generate-help` — HELP.md management
  - `cargo xtask check-openapi` / `generate-openapi` — OpenAPI spec management
- Move openapi.json comparison from curl+python+diff to Rust (serde_json)
- Move generate_help from `examples/` to xtask
- Add `.cargo/config.toml` with xtask alias
- Update CI to use `cargo xtask check` instead of `make check`
- Delete Makefile and `examples/generate_help.rs`

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@sachiniyer sachiniyer merged commit 2e8dbf3 into main Mar 8, 2026
12 checks passed
@sachiniyer sachiniyer deleted the siyer/xtask branch March 8, 2026 15:46
sachiniyer added a commit that referenced this pull request Mar 12, 2026
## Summary
- Bumps version from 0.1.10 to 0.1.11
- The release workflow will auto-tag and build artifacts when merged to
main

### Changes since v0.1.10

**Features:**
- Add shell tab completion via clap_complete (#182)
- Add --scan-id flag to bugs list command (#184)
- Add scans list command (#163)
- Add scans to the skill.md for the detail CLI (#185)
- Auto-detect repository in detail-bugs skill (#160)

**Fixes:**
- Show user-friendly error messages instead of debug output (#129)
- Continue list numbering across pages instead of resetting (#130)
- Add file locking to prevent concurrent config overwrites (#131)

**Chores & Refactoring:**
- Replace Makefile with cargo xtask (#159)
- Upgrade all dependencies to latest versions (#158)
- Enable comprehensive clippy lints (pedantic, nursery, restriction)
(#134, #135, #144, #152, #153, #154, #155)
- Add GitHub Pages workflow for API docs (#178)
- Add Scalar API reference page (#177)
- Add integration tests against live API (#181)
- Remove alpha warning from README (#156)

Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
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.

1 participant