Skip to content
This repository was archived by the owner on Jun 14, 2026. It is now read-only.

feat(DP): add prompt to include Hybrid in release names - #1370

Open
HichamLL04 wants to merge 4 commits into
Audionut:masterfrom
HichamLL04:feat-Hybrid
Open

feat(DP): add prompt to include Hybrid in release names#1370
HichamLL04 wants to merge 4 commits into
Audionut:masterfrom
HichamLL04:feat-Hybrid

Conversation

@HichamLL04

@HichamLL04 HichamLL04 commented May 15, 2026

Copy link
Copy Markdown

If it's on the name, it just adds. If not, ask the user. On --unattended just skips it

#1336

Summary by CodeRabbit

  • Bug Fixes
    • Consistently standardizes release names to normalize "Hybrid" capitalization.
    • Prompts users during interactive runs to add "Hybrid" when appropriate, inserting it near resolution tokens or appending it for clearer names.

Review Change Stack

@github-actions

Copy link
Copy Markdown

Thanks for taking the time to contribute to this project. Upload Assistant is currently in a complete rewrite, and no new development is being conducted on this python source at this time.

If you have come this far, please feel free to leave open, any pull requests regarding new sites being added to the source, as these can serve as the baseline for later conversion.

If your pull request relates to a critical bug, this will be addressed in this code base, and a new release published as needed.

If your pull request only addresses a quite minor bug, it is not likely to be addressed in this code base.

Details for the new code base will follow at a later date.

@coderabbitai

coderabbitai Bot commented May 15, 2026

Copy link
Copy Markdown

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: a17a6fd7-c72b-4835-bb8c-d22124668866

📥 Commits

Reviewing files that changed from the base of the PR and between 6e290fc and c315311.

📒 Files selected for processing (1)
  • src/trackers/DP.py
🚧 Files skipped from review as they are similar to previous changes (1)
  • src/trackers/DP.py

📝 Walkthrough

Walkthrough

The DP tracker now imports re and extends get_name() to derive a technical suffix from the release name by removing configured title and year. The method detects "Hybrid" within that suffix and standardizes its capitalization; if absent, it prompts the user (based on unattended mode) to include "Hybrid" and conditionally inserts it near the resolution token.

Changes

Hybrid Name Normalization

Layer / File(s) Summary
Imports and docstring updates
src/trackers/DP.py
Adds re import and updates the DP class and several method docstrings.
Hybrid name handling in release name
src/trackers/DP.py
Reworks get_name() to compute a technical suffix, detect and normalize "Hybrid" case-insensitively, and when missing in interactive runs, prompt to insert Hybrid around a detected resolution token or append it.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Poem

🐰 I nibbled the name till the suffix was found,
Spotted "hybrid" asleep on the ground,
I raised it to Hybrid, proper and bright,
Or asked with a twitch: "Shall I add it tonight?"
Now release names hop with the right little light. ✨

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The pull request title accurately summarizes the main change: adding a prompt to include 'Hybrid' in release names, which aligns with the detailed changes in the code that add logic to detect, normalize, and prompt for 'Hybrid' inclusion.
Docstring Coverage ✅ Passed Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

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

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant