Skip to content

feat: GitHub Actions should not pull the latest code#85

Merged
kagol merged 2 commits intodevfrom
wyp/yaml-0213
Feb 13, 2026
Merged

feat: GitHub Actions should not pull the latest code#85
kagol merged 2 commits intodevfrom
wyp/yaml-0213

Conversation

@wuyiping0628
Copy link
Collaborator

@wuyiping0628 wuyiping0628 commented Feb 13, 2026

Summary by CodeRabbit

  • Chores
    • Simplified submodule handling in CI/deploy workflows by streamlining initialization and update steps.
  • Style
    • Minor whitespace cleanup in the site theme stylesheet.

@coderabbitai
Copy link

coderabbitai bot commented Feb 13, 2026

Walkthrough

Removed recursive submodule synchronization and remote-merge commands from three GitHub Actions workflows; each workflow now only runs git submodule update --init --recursive and git submodule status --recursive. A minor whitespace line was removed in VitePress theme CSS.

Changes

Cohort / File(s) Summary
Workflow Submodule Updates
.github/workflows/build-docs.yml, .github/workflows/deploy-github.yml, .github/workflows/deploy-obs.yml
Deleted git submodule sync --recursive and git submodule update --remote --merge --recursive. Workflows now rely solely on git submodule update --init --recursive (and status checks) for submodule handling.
Style tweak
.vitepress/theme/style.css
Removed a single blank line between global styles and .dark .VPContent; no selector or property changes.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Poem

🐰
I hopped through CI with careful paws,
Trimmed the sync and merged their laws,
Now submodules wake with simple cheer,
Fewer steps to start the year. ✨

🚥 Pre-merge checks | ✅ 4
✅ Passed checks (4 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately describes the main change: removing commands that pull the latest code from submodule remotes across three GitHub Actions workflows.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Merge Conflict Detection ✅ Passed ✅ No merge conflicts detected when merging into dev

✏️ 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 wyp/yaml-0213

No actionable comments were generated in the recent review. 🎉


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.

@kagol kagol merged commit cee03fb into dev Feb 13, 2026
1 of 2 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.

4 participants