Update gix

[cole-h]

Summary by CodeRabbit

• Chores

  • Updated an internal dependency to a newer compatible version.

• Bug Fixes

  • Refined visibility handling and display: visibility values normalized to "public", "unlisted", and "private", and "hidden" is accepted as a backwards-compatible alias. Adjusted related CLI/options import surface with no behavioral changes.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

Configuration used: defaults

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between e415091 and 2d9d6d4.

📒 Files selected for processing (4)

• src/cli/mod.rs
• src/main.rs
• src/release_metadata.rs
• src/visibility.rs

---

📝 Walkthrough

Walkthrough

Updated project dependency and moved Visibility into a dedicated module: gix bumped 0.71.0 → 0.80.0; introduced src/visibility.rs (pub(crate) Visibility enum + Display); removed inline Visibility from main.rs and updated imports/usages accordingly.

Changes

Cohort / File(s)	Summary
Dependency Update Cargo.toml	Bumped gix dependency from 0.71.0 to 0.80.0, preserving features ["async-network-client","serde"].
Visibility API / Module src/visibility.rs	Added new pub(crate) enum Visibility with serde/clap derives and Display impl; defines variants Public, Unlisted, Hidden (alias), Private.
Callsites / Imports Updated src/main.rs, src/cli/mod.rs, src/release_metadata.rs	Removed embedded Visibility enum from main.rs; updated imports to crate::visibility::Visibility; adjusted struct field types and use statements to reference new module.
Other modules added src/flakehub_auth_fake.rs (new module listed in main.rs mod list)	Module added to module list in main.rs (file contents not shown in diff).

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Poem

🐰 A tiny hop, a tidy refactor song,
Visibility moved where it belongs,
gix bumped up, dependencies bright,
New module nestled snug and right,
I nibble changes — quick and strong. 🥕

🚥 Pre-merge checks | ✅ 1 | ❌ 2

❌ Failed checks (2 warnings)

Check name	Status	Explanation	Resolution
Title check	⚠️ Warning	The PR title 'Update gix' does not reflect the primary changes in the changeset, which include significant refactoring of the Visibility enum (moved to a new module) in addition to the gix dependency update.	Consider updating the title to better represent the main changes, such as 'Refactor Visibility to separate module and update gix' or similar to capture both the dependency update and the structural refactoring.
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (1 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.

_{✏️ 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 update-deps

---

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