Skip to content

[6.19.z] Add Get Host Create View Entity - #2324

Merged
LadislavVasina1 merged 1 commit into
6.19.zfrom
cherry-pick-6.19.z-f6c3167c35c3257ee2fe18f64a35b4137d7c38ca
Feb 26, 2026
Merged

[6.19.z] Add Get Host Create View Entity#2324
LadislavVasina1 merged 1 commit into
6.19.zfrom
cherry-pick-6.19.z-f6c3167c35c3257ee2fe18f64a35b4137d7c38ca

Conversation

@Satellite-QE

Copy link
Copy Markdown
Contributor

Cherrypick of PR: #2323

Needed by: SatelliteQE/robottelo#20888 (PRT ran and passed here)

(cherry picked from commit f6c3167)
@Satellite-QE Satellite-QE added 6.19.z Auto_Cherry_Picked GHA has automatically cherrypicked this PR No-CherryPick PR doesnt need CherryPick to previous branches labels Feb 26, 2026

@sourcery-ai sourcery-ai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Hey - I've left some high level feedback:

  • Consider renaming get_create_form to something like get_create_view (or updating the docstring) to better reflect that it returns a HostCreateView rather than a form object, keeping naming consistent and self-documenting.
Prompt for AI Agents
Please address the comments from this code review:

## Overall Comments
- Consider renaming `get_create_form` to something like `get_create_view` (or updating the docstring) to better reflect that it returns a `HostCreateView` rather than a form object, keeping naming consistent and self-documenting.

Sourcery is free for open source - if you like our reviews please consider sharing them ✨
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.

@LadislavVasina1
LadislavVasina1 merged commit e8902ab into 6.19.z Feb 26, 2026
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

6.19.z Auto_Cherry_Picked GHA has automatically cherrypicked this PR No-CherryPick PR doesnt need CherryPick to previous branches

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants