Skip to content

refactor: move httpmock and serde_json to dev-dependencies#147

Merged
joaolfp merged 1 commit into
mainfrom
refactor/move-test-deps-to-dev-dependencies
Apr 28, 2026
Merged

refactor: move httpmock and serde_json to dev-dependencies#147
joaolfp merged 1 commit into
mainfrom
refactor/move-test-deps-to-dev-dependencies

Conversation

@joaolfp
Copy link
Copy Markdown
Contributor

@joaolfp joaolfp commented Apr 28, 2026

✨ Summary

  • move httpmock and serde_json from regular dependencies to [dev-dependencies] in Cargo.toml
  • update Cargo.lock with minor version bumps (async-lock 3.4.1 → 3.4.2, typenum 1.19.0 → 1.20.0)

🔧 Type of Change

  • ✨ Enhancement
  • 🐞 Bug fix
  • 🔐 Security fix
  • 💥 Breaking change
  • 🚀 New feature
  • 📦 New release
  • 📚 Documentation
  • ♻️ Refactor

Note

Low Risk
Low risk dependency-scope change that should only affect build/test compilation; runtime code paths are unchanged aside from minor indirect crate bumps via Cargo.lock.

Overview
Moves httpmock and serde_json out of main [dependencies] into [dev-dependencies], reducing the release build dependency set.

Regenerates Cargo.lock, picking up small indirect updates (async-lock 3.4.13.4.2, typenum 1.19.01.20.0).

Reviewed by Cursor Bugbot for commit e1fd24c. Bugbot is set up for automated code reviews on this repo. Configure here.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@github-actions
Copy link
Copy Markdown

🎉 All unit tests passed!

@joaolfp joaolfp merged commit 9bbfd5d into main Apr 28, 2026
4 checks passed
@joaolfp joaolfp deleted the refactor/move-test-deps-to-dev-dependencies branch April 28, 2026 04:22
Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request updates the versions of async-lock and typenum in Cargo.lock. Additionally, it moves httpmock and serde_json from the main dependencies to dev-dependencies in Cargo.toml. I have no feedback to provide.

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