Skip to content

docs: fix typos in v1 to v2 migration guide - #876

Merged
mabuyo merged 1 commit into
mainfrom
docs/fix-v2-migration-guide
Oct 7, 2026
Merged

mabuyo merged 1 commit into
mainfrom
docs/fix-v2-migration-guide

Conversation

@DaleSeo

@DaleSeo DaleSeo commented Oct 7, 2026 •

Copy link
Copy Markdown
Member

🧹

@apollo-librarian

apollo-librarian Bot commented Oct 7, 2026 •

Copy link
Copy Markdown
Contributor

✅ Docs preview ready

The preview is ready to be viewed. View the preview

File Changes

1 new, 3 changed, 0 removed
+ (developer-tools)/apollo-mcp-server/(latest)/migrating-from-v1.mdx
* (developer-tools)/apollo-mcp-server/(latest)/config-file.mdx
* (developer-tools)/apollo-mcp-server/(latest)/run.mdx
* (developer-tools)/apollo-mcp-server/(latest)/_sidebar.yaml

Build ID: 44a3ce5a3347ba00cc80675c
Build Logs: View logs

URL: https://www.apollographql.com/docs/deploy-preview/44a3ce5a3347ba00cc80675c


✅ AI Style Review — No Issues Found

The pull request does not have any style issues.

Duration: 2296ms
Review Log: View detailed log

This review is AI-generated. Please use common sense when accepting these suggestions, as they may not always be accurate or appropriate for your specific context.

@coderabbitai

coderabbitai Bot commented Oct 7, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration
  • Configuration used: Repository UI
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: d24e29a7-23c7-4cf0-b25e-0259c228fa48
📥 Commits

Reviewing files that changed from the base of the PR and between e2ebd2b and ea2bd59.

📒 Files selected for processing (1)
  • docs/source/migrating-from-v1.mdx
💤 Files with no reviewable changes (1)
  • docs/source/migrating-from-v1.mdx

Included review availability: This review used your included allowance. Your plan provides up to 2 included reviews per hour; 1 remain after this review.


📝 Summary

Summary by CodeRabbit

  • Documentation
    • Removed a duplicate migration-table entry and an empty bullet from the v2.0 feature list.

Walkthrough

The migration guide no longer contains a duplicate authorization_servers migration row or an empty item in the v2.0 feature list.

Changes

Migration guide cleanup

Layer / File(s) Summary
Remove duplicate and empty entries
docs/source/migrating-from-v1.mdx
The migration table retains one authorization_servers entry. The v2.0 feature list no longer has an empty bullet.

Priority: ⬇️ Low

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

Change: Other

Suggested reviewers: dcwalter

Merge Risk: ⚪ Minimal · up to ea2bd

The migration guide cleanup is ready to merge after normal checks.

🚥 Pre-merge checks | ✅ 4 | ❓ 1

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Description check ❓ Inconclusive The cleanup emoji suggests a documentation cleanup, but the description does not explain what changed. Add a brief description stating that the change removes a duplicate authorization_servers row and an empty bullet from the migration guide.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title identifies the v1-to-v2 migration guide and a documentation cleanup, but it calls the changes typo fixes. The changes remove a duplicate table row and an empty bullet.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
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.
  • Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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.

@DaleSeo
DaleSeo marked this pull request as ready for review October 7, 2026 03:12
@DaleSeo
DaleSeo requested review from a team as code owners October 7, 2026 03:12
@DaleSeo DaleSeo changed the title docs: fix gaps in v1 to v2 migration guide docs: fix typos in v1 to v2 migration guide Oct 7, 2026

@claude claude Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM, trivial documentation cleanup. What was reviewed: confirmed the diff removes exactly one duplicate table row (the authorization_servers no longer normalized entry, identical content) and one stray empty bullet (- ) in the "What's new in v2.0" list, with no other content changes.

Extended reasoning...

The change touches only docs/source/migrating-from-v1.mdx, a documentation file with no security-sensitive surface. It is a minimal, mechanical cleanup (deleting a duplicated row and an empty bullet) verified directly against the diff, so it's simple and obviously correct enough to not need human review.

@mabuyo
mabuyo merged commit 04e4e6e into main Oct 7, 2026
16 checks passed
@mabuyo
mabuyo deleted the docs/fix-v2-migration-guide branch October 7, 2026 03:36
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