Skip to content

chore: update codeowners to use new install team handle#393

Merged
rm3l merged 1 commit into
redhat-developer:mainfrom
Jdubrick:update-codeowners-may13
May 13, 2026
Merged

chore: update codeowners to use new install team handle#393
rm3l merged 1 commit into
redhat-developer:mainfrom
Jdubrick:update-codeowners-may13

Conversation

@Jdubrick
Copy link
Copy Markdown

Description of the change

  • CODEOWNERS file was using rhdh-helm as the global owner, but that team does not exist anymore under @redhat-developer, it looks like rhdh-install is the new one

Which issue(s) does this PR fix or relate to

N/A

How to test changes / Special notes to the reviewer

Checklist

  • For each Chart updated, version bumped in the corresponding Chart.yaml according to Semantic Versioning.
  • For each Chart updated, variables are documented in the values.yaml and added to the corresponding README.md. The pre-commit utility can be used to generate the necessary content. Run pre-commit run --all-files to run the hooks and then push any resulting changes. The pre-commit Workflow will enforce this and warn you if needed.
  • JSON Schema template updated and re-generated the raw schema via the pre-commit hook.
  • Tests pass using the Chart Testing tool and the ct lint command.
  • If you updated the orchestrator-infra chart, make sure the versions of the Knative CRDs are aligned with the versions of the CRDs installed by the OpenShift Serverless operators declared in the values.yaml file. See Installing Knative Eventing and Knative Serving CRDs for more details.

Signed-off-by: Jordan Dubrick <jdubrick@redhat.com>
@openshift-ci openshift-ci Bot requested review from rm3l and zdrapela May 13, 2026 14:26
@rhdh-qodo-merge
Copy link
Copy Markdown

rhdh-qodo-merge Bot commented May 13, 2026

Code Review by Qodo

🐞 Bugs (0) 📘 Rule violations (0) 📎 Requirement gaps (0)

Grey Divider

Great, no issues found!

Qodo reviewed your code and found no material issues that require review

Grey Divider

Qodo Logo

@sonarqubecloud
Copy link
Copy Markdown

@rhdh-qodo-merge
Copy link
Copy Markdown

Review Summary by Qodo

Update CODEOWNERS to use rhdh-install team handle

📦 Other

Grey Divider

Walkthroughs

Description
• Update global CODEOWNERS rule to use new team handle
• Replace deprecated rhdh-helm with rhdh-install team
Diagram
flowchart LR
  old["@redhat-developer/rhdh-helm"] -- "deprecated team" --> update["Update CODEOWNERS"]
  update -- "new team handle" --> new["@redhat-developer/rhdh-install"]
Loading

Grey Divider

File Changes

1. .github/CODEOWNERS ⚙️ Configuration changes +1/-1

Update global CODEOWNERS team reference

• Updated global owner pattern from @redhat-developer/rhdh-helm to
 @redhat-developer/rhdh-install
• Reflects team restructuring where the old helm team no longer exists

.github/CODEOWNERS


Grey Divider

Qodo Logo

@Jdubrick
Copy link
Copy Markdown
Author

/cc @gazarenkov

@openshift-ci openshift-ci Bot requested a review from gazarenkov May 13, 2026 14:34
Copy link
Copy Markdown
Member

@rm3l rm3l left a comment

Choose a reason for hiding this comment

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

Good catch. Thanks.

@openshift-ci openshift-ci Bot added the lgtm label May 13, 2026
@rm3l rm3l merged commit 6b19f37 into redhat-developer:main May 13, 2026
5 of 6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants