Update source branch refs from master to main#51
Conversation
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>
WalkthroughThis pull request updates three configuration and build files to reference the Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Poem
🚥 Pre-merge checks | ✅ 2 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (2 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
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. Comment |
There was a problem hiding this comment.
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
📒 Files selected for processing (3)
prototypes/docusaurus/docusaurus.config.tsprototypes/docusaurus/sidebars.tsscripts/sync-docs.mjs
|
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>
Summary
mastertomaineditUrlindocusaurus.config.tsto point tomainsidebars.tsto usemainsync-docs.mjsfrommastertomainWithout this fix, "Edit this page" links, the changelog link, and doc syncing all point to the now-stale
masterbranch.Test plan
npm run sync-docsand confirm it clones from themainbranch🤖 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
mastertomainfor the upstreamreact_on_railsrepo.This changes the GitHub Actions site build/deploy workflow to sync docs from
main, updates DocusauruseditUrland the sidebar changelog link to point atmain, and adjustsscripts/sync-docs.mjsto default to cloningmainwhen 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
mainbranch instead of themasterbranch across configuration and synchronization scripts.