Skip to content

SAT-45979 errata sync status - #2485

Merged
sambible merged 1 commit into
SatelliteQE:masterfrom
vijaysawant:SAT-45979-Errata-sync-status
Jul 22, 2026
Merged

SAT-45979 errata sync status#2485
sambible merged 1 commit into
SatelliteQE:masterfrom
vijaysawant:SAT-45979-Errata-sync-status

Conversation

@vijaysawant

Copy link
Copy Markdown
Contributor

Add search_repositories method to ErrataEntity to support filtering errata repositories by lifecycle environment and content view. This enables UI test coverage for SAT-45979, which verifies errata sync status is correctly displayed across content views.

Robottelo PR SatelliteQE/robottelo#22196

rh-pre-commit.version: 2.3.2
rh-pre-commit.check-secrets: ENABLED
@vijaysawant vijaysawant added CherryPick PR needs CherryPick to previous branches AutoMerge_Cherry_Picked Automatically merge the PR is PRT and all checks are passing Stream 6.18.z 6.19.z labels Jul 21, 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:

  • The docstring parameter annotations for search_repositories are slightly off (:param str optional lce: / cv:); consider using the standard Sphinx style (e.g., :param str lce:) and describing the optionality in the text to keep generated docs and IDE hints clean.
  • For consistency and readability, consider using more descriptive parameter names than lce and cv (e.g., lifecycle_env and content_view), matching any existing naming conventions for these concepts elsewhere in the codebase.
Prompt for AI Agents
Please address the comments from this code review:

## Overall Comments
- The docstring parameter annotations for `search_repositories` are slightly off (`:param str optional lce:` / `cv:`); consider using the standard Sphinx style (e.g., `:param str lce:`) and describing the optionality in the text to keep generated docs and IDE hints clean.
- For consistency and readability, consider using more descriptive parameter names than `lce` and `cv` (e.g., `lifecycle_env` and `content_view`), matching any existing naming conventions for these concepts elsewhere in the codebase.

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.

@vsedmik vsedmik 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.

Looks good to me

@vijaysawant vijaysawant added No-CherryPick PR doesnt need CherryPick to previous branches and removed CherryPick PR needs CherryPick to previous branches AutoMerge_Cherry_Picked Automatically merge the PR is PRT and all checks are passing 6.18.z 6.19.z labels Jul 22, 2026
@sambible
sambible merged commit 53dab70 into SatelliteQE:master Jul 22, 2026
18 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

No-CherryPick PR doesnt need CherryPick to previous branches Stream

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants