chore: Fix remaining issue in rhoai-2.25

[ysok]

Summary

• Fix remaining issue on rhoai-2.25

Test plan

• Verify CI on the PR

Summary by CodeRabbit

• Documentation
  • Added a new top-level “Test” heading to the README.

[openshift-ci]

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please assign jesuino for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found here.

Details

Needs approval from an approver in each of these files:

• OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[github-actions]

@ysok — This PR is from a fork.
The build-rhoai CI job was skipped because subscription
builds (RHEL, AIPCC) need secrets unavailable to forks.
ODH builds and code quality checks still ran.

Recommended: Push your branch to the main repo for full CI:

git remote add upstream https://github.com/red-hat-data-services/notebooks.git
git push upstream HEAD:ysok/your-branch-name

Then open a new PR from that branch.

No push access? A maintainer will cherry-pick and test your changes.

See CONTRIBUTING.md for details.

[coderabbitai]

📝 Walkthrough

Walkthrough

A new top-level # Test heading is added as the first line of README.md.

Changes

README update

Layer / File(s)	Summary
Add README heading README.md	Adds # Test at the beginning of the document.

Estimated code review effort: 1 (Trivial) | ~2 minutes

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Description check	⚠️ Warning	The description omits required template sections, detailed testing, checklist items, and merge criteria.	Add Description and How Has It Been Tested sections with environment/test details, complete the self-checklist, and mark applicable merge criteria.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title is relevant to the branch fix, though it is broader than the specific README change.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
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.

✨ 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

• X
• Mastodon
• Reddit
• LinkedIn

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

[codecov-commenter]

⚠️ Please install the to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

✅ All modified and coverable lines are covered by tests.
⚠️ Please upload report for BASE (rhoai-2.25@e527591). Learn more about missing BASE report.
✅ All tests successful. No failed tests found.
❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files

@@             Coverage Diff              @@
##             rhoai-2.25   #2532   +/-   ##
============================================
  Coverage              ?       0           
============================================
  Files                 ?       0           
  Lines                 ?       0           
  Branches              ?       0           
============================================
  Hits                  ?       0           
  Misses                ?       0           
  Partials              ?       0           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[ysok]

/build-konflux

[coderabbitai]

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@README.md`:
- Line 1: Replace the new top-level “Test” heading in README.md with a
second-level heading if it is intended as a section, or remove it if
unnecessary; preserve the existing “OpenDataHub Notebooks” H1 as the document’s
sole top-level heading.

🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

• Push a commit to this branch (recommended)
• Create a new PR with the fixes

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Enterprise

Run ID: 4d18d441-2af8-46b9-a661-b72520585ec7

📥 Commits

Reviewing files that changed from the base of the PR and between e527591 and 46fce51.

📒 Files selected for processing (1)

• README.md

[coderabbitai]

📐 Maintainability & Code Quality | 🟡 Minor | ⚡ Quick win

Avoid introducing a second top-level heading.

README.md already has # OpenDataHub Notebooks on Line 2, so # Test creates competing H1 titles and makes the generic heading appear to be the document title. Use ## Test if this is intended as a section, or remove it.

🧰 Tools

🪛 GitHub Check: Generate job matrix

[warning] 1-1: Fork PR — subscription builds will fail
Push your branch to the main repo for full CI. See CONTRIBUTING.md.

🤖 Prompt for AI Agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@README.md` at line 1, Replace the new top-level “Test” heading in README.md
with a second-level heading if it is intended as a section, or remove it if
unnecessary; preserve the existing “OpenDataHub Notebooks” H1 as the document’s
sole top-level heading.

[atheo89]

/build-konflux