Skip to content

Release v0.3.51#217

Merged
bradhe merged 2 commits intomainfrom
develop
Mar 2, 2026
Merged

Release v0.3.51#217
bradhe merged 2 commits intomainfrom
develop

Conversation

@bradhe
Copy link
Contributor

@bradhe bradhe commented Mar 2, 2026

  • Pass hidden parameters through to the app manifest
  • Bump version to v0.3.51

Summary by CodeRabbit

  • New Features

    • Parameters can now be marked as hidden in package definitions. Users can designate specific parameters to remain hidden, providing enhanced control over visibility and streamlining parameter management. This addition offers greater flexibility for package configuration workflows.
  • Chores

    • Version bumped to 0.3.51

* Pump hidden through to manifest

* Add a quick test for this change
@coderabbitai
Copy link

coderabbitai bot commented Mar 2, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Disabled knowledge base sources:

  • Linear integration is disabled

You can enable these sources in your CodeRabbit configuration.

📥 Commits

Reviewing files that changed from the base of the PR and between 492056d and c41a2cb.

⛔ Files ignored due to path filters (2)
  • Cargo.lock is excluded by !**/*.lock
  • uv.lock is excluded by !**/*.lock
📒 Files selected for processing (4)
  • Cargo.toml
  • crates/tower-package/src/lib.rs
  • crates/tower-package/tests/package_test.rs
  • pyproject.toml

📝 Walkthrough

Walkthrough

This PR bumps the workspace and project version to 0.3.51, adds a new hidden: bool field to the Parameter struct with serde default annotation, propagates this field in parameter mapping logic, and introduces a test validating that both visible and hidden parameters are included in package manifests.

Changes

Cohort / File(s) Summary
Version Bumps
Cargo.toml, pyproject.toml
Updated workspace and project version from 0.3.50 to 0.3.51.
Parameter Enhancement
crates/tower-package/src/lib.rs
Added new public field hidden: bool to Parameter struct with #[serde(default)] and updated get_parameters to propagate this field when mapping parameters.
Tests & Exports
crates/tower-package/tests/package_test.rs
Exported Parameter struct and added test it_includes_hidden_parameters_in_manifest validating that both visible and hidden parameters appear in package manifests. Note: Test appears duplicated in diff.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Suggested reviewers

  • jo-sm
  • giray123

Poem

🐰 A hidden field hops into place,
With serde defaults keeping pace,
Parameters now wear secret cloaks,
While tests ensure no hidden jokes! 🥕✨

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 50.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'Release v0.3.51' accurately reflects the main objectives of the PR: bumping the version to 0.3.51 and implementing the hidden parameters feature.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
  • 📝 Generate docstrings (stacked PR)
  • 📝 Generate docstrings (commit on current branch)
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch develop

Comment @coderabbitai help to get the list of available commands and usage tips.

@coderabbitai coderabbitai bot requested review from giray123 and jo-sm March 2, 2026 16:38
@bradhe bradhe merged commit 48d1fb5 into main Mar 2, 2026
31 checks passed
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