Skip to content

Comments

Warehouse Migrations: Hide content until section will be re-populated#545

Merged
bmunkholm merged 1 commit intomainfrom
orphan-rockset
Feb 9, 2026
Merged

Warehouse Migrations: Hide content until section will be re-populated#545
bmunkholm merged 1 commit intomainfrom
orphan-rockset

Conversation

@amotl
Copy link
Member

@amotl amotl commented Feb 9, 2026

About

Optional: remove Rockset migration entirely? Likely no one is left to migrate.

Yes, it's a bit lonely. Let's rework and expand it later, so hide it for now?

Preview

References

@coderabbitai
Copy link

coderabbitai bot commented Feb 9, 2026

Caution

Review failed

The pull request is closed.

Walkthrough

Adds YAML front matter orphan: true to two migrate docs and adds a :class-item: visually-hidden front-matter directive to a Warehouse Migrations item in the overview page; no content or control-flow changes.

Changes

Cohort / File(s) Summary
Orphan Metadata Additions
docs/integrate/category/migrate/index.md, docs/integrate/category/migrate/rockset/index.md
Inserted YAML front matter with orphan: true to mark these pages as orphans.
Overview Grid Item Visibility
docs/integrate/category/overview.md
Added :class-item: visually-hidden front-matter directive to the Warehouse Migrations grid item.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

Suggested labels

maintenance, reorganize, refurbish

Suggested reviewers

  • kneth

Poem

🐰 I hop through pages, quiet and spry,
Tucking orphan tags where hidden things lie.
A furtive class makes a grid item small,
Metadata whispers — tidy one and all.

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and specifically describes the main change: hiding warehouse migrations content until the section is re-populated, which aligns with the changes adding orphan directives and visually-hidden classes.
Description check ✅ Passed The description is related to the changeset, noting the intent to hide Rockset migration content and providing context about potentially removing it entirely, which corresponds to the markup changes in the PR.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch orphan-rockset

Warning

Review ran into problems

🔥 Problems

Git: Failed to clone repository. Please run the @coderabbitai full review command to re-trigger a full review. If the issue persists, set path_filters to include or exclude specific files.


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.

@amotl amotl requested review from bmunkholm and kneth February 9, 2026 16:30
@amotl amotl marked this pull request as ready for review February 9, 2026 16:34
@bmunkholm bmunkholm merged commit 2a08080 into main Feb 9, 2026
0 of 3 checks passed
@bmunkholm bmunkholm deleted the orphan-rockset branch February 9, 2026 18:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants