Skip to content

Update source branch refs from master to main#51

Merged
justin808 merged 2 commits intomainfrom
jg/fix-toc-sidebar
Mar 22, 2026
Merged

Update source branch refs from master to main#51
justin808 merged 2 commits intomainfrom
jg/fix-toc-sidebar

Conversation

@justin808
Copy link
Member

@justin808 justin808 commented Mar 22, 2026

Summary

  • The react_on_rails repo renamed its default branch from master to main
  • Updates the editUrl in docusaurus.config.ts to point to main
  • Updates the changelog link in sidebars.ts to use main
  • Updates the default clone ref in sync-docs.mjs from master to main

Without this fix, "Edit this page" links, the changelog link, and doc syncing all point to the now-stale master branch.

Test plan

  • Verify "Edit this page" links on doc pages resolve correctly
  • Verify the "Full Changelog (GitHub)" sidebar link works
  • Run npm run sync-docs and confirm it clones from the main branch

🤖 Generated with Claude Code


Note

Low Risk
Low risk: this only updates branch names used for doc sync and external GitHub links, with no runtime logic changes beyond which ref is cloned/linked.

Overview
Updates all docs-related references from master to main for the upstream react_on_rails repo.

This changes the GitHub Actions site build/deploy workflow to sync docs from main, updates Docusaurus editUrl and the sidebar changelog link to point at main, and adjusts scripts/sync-docs.mjs to default to cloning main when no local docs repo is present.

Written by Cursor Bugbot for commit 7041f67. This will update automatically on new commits. Configure here.

Summary by CodeRabbit

  • Chores
    • Updated documentation references to use the main branch instead of the master branch across configuration and synchronization scripts.

The react_on_rails repo renamed its default branch from master to main.
Update editUrl, changelog link, and sync-docs clone ref accordingly.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@coderabbitai
Copy link

coderabbitai bot commented Mar 22, 2026

Walkthrough

This pull request updates three configuration and build files to reference the main Git branch instead of master across docusaurus configuration, sidebar links, and documentation synchronization scripts.

Changes

Cohort / File(s) Summary
Docusaurus Configuration
prototypes/docusaurus/docusaurus.config.ts, prototypes/docusaurus/sidebars.ts
Updated GitHub branch references from master to main in the editUrl generator and CHANGELOG link.
Build Scripts
scripts/sync-docs.mjs
Changed default git reference fallback from master to main for the react_on_rails documentation repository clone operation.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 From master to main we hop and bound,
Three little changes, all so sound,
Branches renamed, a cleaner sight,
Our documents dance in the main branch light,
Simple, swift, and oh so right! 🌳

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The PR title directly and accurately summarizes the primary change: updating branch references from 'master' to 'main' across three files.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.

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

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch jg/fix-toc-sidebar

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.

Copy link

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@scripts/sync-docs.mjs`:
- Line 118: CI still forces REACT_ON_RAILS_REF=master so your new fallback to
"main" is ignored; update the workflow to align with the code change by changing
the REACT_ON_RAILS_REF environment value from "master" to "main" in both Sync
docs steps (the Sync docs step in the build job and the Sync docs step in the
deploy job) so the automated sync uses the "main" branch when the env is set in
the script.
🪄 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: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: e465fad1-b27b-4972-8a6c-5b65e0fa53d2

📥 Commits

Reviewing files that changed from the base of the PR and between 6d18b92 and 747f4d9.

📒 Files selected for processing (3)
  • prototypes/docusaurus/docusaurus.config.ts
  • prototypes/docusaurus/sidebars.ts
  • scripts/sync-docs.mjs

@github-actions
Copy link

github-actions bot commented Mar 22, 2026

Cloudflare preview deployed.

Aligns the workflow environment variable with the sync-docs script's
updated default branch reference.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@justin808 justin808 merged commit 41480e1 into main Mar 22, 2026
4 checks passed
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.

1 participant