Skip to content

Conversation

amotl
Copy link
Member

@amotl amotl commented Oct 6, 2025

About

All the Markdown has been migrated to the CrateDB Guide / Handbook.

-- https://community.cratedb.com/t/overview-of-cratedb-integration-tutorials/1015

References

Copy link

coderabbitai bot commented Oct 6, 2025

Warning

Rate limit exceeded

@amotl has exceeded the limit for the number of commits or files that can be reviewed per hour. Please wait 8 minutes and 52 seconds before requesting another review.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

📥 Commits

Reviewing files that changed from the base of the PR and between e04a151 and 6037624.

📒 Files selected for processing (3)
  • README.md (0 hunks)
  • docs/integrate/index.md (0 hunks)
  • docs/solution/migrate/rockset/index.md (2 hunks)

Walkthrough

Documentation updates: removed outdated integration tutorial references, cleaned up a tip block and link definitions, and adjusted homepage and Rockset migration page content to point to Integration Guides and added a brief feature blurb.

Changes

Cohort / File(s) Summary of changes
Homepage updates
docs/index.md
Removed a Markdown link reference to the Overview of CrateDB integration tutorials; added a line to the “All features” card.
Integrations index cleanup
docs/integrate/index.md
Removed a tip admonition referencing the Overview of CrateDB integration tutorials and deleted the associated reference link definition.
Rockset migration links
docs/solution/migrate/rockset/index.md
Replaced “Integration Tutorials I/II” links with “Integration Guides” and updated reference anchors accordingly.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related issues

Possibly related PRs

Suggested reviewers

  • kneth
  • bmunkholm
  • hammerhead

Poem

A nibble of links, a hop through docs,
I tightened the trails and cleared the blocks.
Tutorials retired, guides take the lead—
carrots of clarity for every need. 🥕
With whiskers twitching, I merge and beam,
our documentation now sleek and clean.

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Title Check ✅ Passed The title clearly summarizes the principal change by specifying the removal of the "Overview of CrateDB integration tutorials" reference in a concise, specific manner without extraneous details, matching the documentation cleanup focus of the changeset.
Docstring Coverage ✅ Passed No functions found in the changes. Docstring coverage check skipped.
Description Check ✅ Passed The pull request description states that all Markdown has been migrated to the CrateDB Guide / Handbook and includes the relevant overview link, which directly corresponds to the removal of those references in the documented changes. This provides clear context about why the links were removed and aligns with the modifications in the 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 added sanding-1200 Fine sanding. cross linking Linking to different locations of the documentation. maintenance C'est la vie. and removed sanding-1200 Fine sanding. labels Oct 6, 2025
@amotl amotl marked this pull request as ready for review October 6, 2025 02:46
@amotl amotl requested review from bmunkholm, kneth and surister October 6, 2025 02:46
Copy link
Contributor

@bmunkholm bmunkholm left a comment

Choose a reason for hiding this comment

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

Should we post an note/reference to this in the Forum then?

@amotl
Copy link
Member Author

amotl commented Oct 6, 2025

Should we post a note/reference to this in the Forum then?

The community forum resources need to be updated, right. Example: https://community.cratedb.com/t/autoscale-a-cratedb-cloud-cluster/1731.

@coderabbitai: Please add this to GH-102, thanks.

Copy link

coderabbitai bot commented Oct 6, 2025

Tip

For best results, initiate chat on the files or code changes.

🧩 Analysis chain

🏁 Script executed:

#!/bin/bash
# Read issue #102 to understand its context and current content
gh issue view 102 --repo crate/cratedb-guide

Length of output: 556


🏁 Script executed:

#!/bin/bash
# Add a comment to issue #102 about updating community forum resources
gh issue comment 102 --repo crate/cratedb-guide --body "## Update Community Forum Resources

The community forum resources need to be updated to reflect the consolidation of integration guides.

**Example forum post that needs updating:**
- https://community.cratedb.com/t/autoscale-a-cratedb-cloud-cluster/1731

**Context:**
PR #370 removes the \"Overview of CrateDB integration tutorials\" reference, as the content has been migrated to the CrateDB Guide/Handbook. Forum posts should be reviewed and updated accordingly to point to the consolidated integration guides.

Related: https://github.com/crate/cratedb-guide/pull/370"

Length of output: 73


@amotl: Done! I've added a comment to issue #102 about updating the community forum resources. The comment includes the example forum post you mentioned and provides context about PR #370 removing the "Overview of CrateDB integration tutorials" reference.

@amotl amotl force-pushed the rm-integration-tutorials branch from e04a151 to 4d799d2 Compare October 6, 2025 20:50
@amotl amotl force-pushed the rm-integration-tutorials branch from 4d799d2 to 6037624 Compare October 6, 2025 20:51
@amotl amotl merged commit cc577ee into main Oct 6, 2025
3 checks passed
@amotl amotl deleted the rm-integration-tutorials branch October 6, 2025 20:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cross linking Linking to different locations of the documentation. maintenance C'est la vie.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants