Skip to content

Comments

build(deps): remove dev-dependencies#60

Merged
Nugine merged 4 commits intomainfrom
build/remove-dev-dependencies
Jan 20, 2026
Merged

build(deps): remove dev-dependencies#60
Nugine merged 4 commits intomainfrom
build/remove-dev-dependencies

Conversation

@Nugine
Copy link
Owner

@Nugine Nugine commented Jan 20, 2026

No description provided.

Copilot AI review requested due to automatic review settings January 20, 2026 13:31
@codecov
Copy link

codecov bot commented Jan 20, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 94.86%. Comparing base (35e1034) to head (2d08eba).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #60   +/-   ##
=======================================
  Coverage   94.86%   94.86%           
=======================================
  Files          37       37           
  Lines        3855     3855           
  Branches     3855     3855           
=======================================
  Hits         3657     3657           
  Misses        171      171           
  Partials       27       27           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR refactors the dependency management structure by removing dev-dependencies and converting them to optional workspace dependencies. The changes centralize dependency version management at the workspace level and properly gate tests that require optional features.

Changes:

  • Introduced workspace-level dependency management for heck, http, and regex in the root Cargo.toml
  • Converted dev-dependencies to optional regular dependencies with proper feature gating
  • Added feature gate to test that depends on the heck crate to ensure it only runs when the case feature is enabled

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated no comments.

Show a summary per file
File Description
Cargo.toml Added workspace dependencies section with heck, http, and regex versions
crates/const-str/Cargo.toml Removed dev-dependencies section, converted dependencies to workspace references, updated feature definitions to include dep: prefix, reordered metadata section
crates/const-str-proc-macro/Cargo.toml Updated dependencies to use workspace references and sorted them alphabetically
crates/const-str/src/__ctfe/ascii_case.rs Added #[cfg(feature = "case")] gate to test function that uses heck crate
.github/workflows/ci.yml Removed codecov token field from workflow configuration

@Nugine Nugine merged commit 63dd158 into main Jan 20, 2026
13 checks passed
@Nugine Nugine deleted the build/remove-dev-dependencies branch January 20, 2026 13:36
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