Skip to content

remove pinging website - #24

Merged
batuhan merged 1 commit into
mainfrom
batuhan/hook
Aug 19, 2026
Merged

remove pinging website#24
batuhan merged 1 commit into
mainfrom
batuhan/hook

Conversation

@batuhan

@batuhan batuhan commented Aug 19, 2026

Copy link
Copy Markdown
Contributor

View with [code]smith Autofix with [code]smith
Need help on this PR? Tag @codesmith-bot with what you need. Autofix is disabled.

Summary by CodeRabbit

  • Chores
    • Removed the automated notification that triggered website publishing after documentation updates.
    • Documentation changes will no longer send cross-repository update events.
@coderabbitai

coderabbitai Bot commented Aug 19, 2026

Copy link
Copy Markdown

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: eae2657a-f36f-4b05-855c-97d5712a8cba

📥 Commits

Reviewing files that changed from the base of the PR and between b7eb03d and 476034b.

📒 Files selected for processing (1)
  • .github/workflows/ci.yml
💤 Files with no reviewable changes (1)
  • .github/workflows/ci.yml

Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review.


📝 Walkthrough

Walkthrough

The CI workflow removes the notify-website job and its cross-repository docs-updated dispatch.

Changes

Cohort / File(s) Summary
CI workflow
.github/workflows/ci.yml
Removes GitHub App token creation and the conditional website notification dispatch to spacefast/www.

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

Merge Risk: ⚪ Minimal · up to 47603

This PR removes website-pinging workflow behavior, and no actionable merge-blocking risk remains based on the supplied evidence.

Possibly related PRs

  • spacefast/docs#11: Added the website rebuild notification mechanism removed by this PR.
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly describes the removal of the website notification job from the CI workflow.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
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.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch batuhan/hook

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.

@batuhan
batuhan merged commit f2d5a61 into main Aug 19, 2026
5 checks passed
@batuhan
batuhan deleted the batuhan/hook branch August 19, 2026 20:04
batuhan added a commit that referenced this pull request Aug 29, 2026
Removing notify-website on Aug 19 severed the only path that advances
.spacefast/docs-revision in spacefast/www: the Refresh Docs workflow
there fires solely on this repository_dispatch, so the pin froze at
b7eb03d and every Website build since has composed ten-day-old Docs.
Reverts #24.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

1 participant