Skip to content

chore: freeze the fork and drop the weekly upstream sync - #41

Merged
kixelated merged 1 commit into
mainfrom
chore/freeze
Oct 16, 2026
Merged

kixelated merged 1 commit into
mainfrom
chore/freeze

Conversation

@kixelated

Copy link
Copy Markdown

moq-dev/moq replaced this fork with its in-tree rs/moq-quic, a hard fork of quinn, in moq-dev/moq#5664. This marks the fork frozen.

  • Delete .github/workflows/moq-sync.yml. It has failed every scheduled run on a workflow-permission error, and the fork no longer merges n0-computer/noq.
  • README: frozen to security patches for MoQ's release line, archived once no released MoQ crate depends on it, successor rs/moq-quic in moq-dev/moq.
  • moq-ci.yml: drop the header's sync rationale for keeping upstream's workflows.

No crate, API, or wire change. Archiving the repository is a separate, later decision.

(Written by Claude Opus 5.5)

🤖 Generated with Claude Code

moq-dev/moq now runs on its in-tree moq-quic, a hard fork of quinn, so this
fork only takes security patches for MoQ's release line until no released
MoQ crate depends on it.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Oct 16, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-10-16T16:09:52.506667Z 6db2b8a PR opened
🔒 Security Review ✅ Completed 2026-10-16T16:11:13.898472Z 6db2b8a PR opened
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@coderabbitai

coderabbitai Bot commented Oct 16, 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: Organization UI
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 913e93b5-2b65-4ec1-8a8f-e21741775755
📥 Commits

Reviewing files that changed from the base of the PR and between e27c0ce and 6db2b8a.

📒 Files selected for processing: 3
  • .github/workflows/moq-ci.yml
  • .github/workflows/moq-sync.yml
  • README.md
💤 Files with no reviewable changes: 1
  • .github/workflows/moq-sync.yml

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

Walkthrough

The README now describes the fork as frozen, identifies the successor repository, and clarifies crate versioning and release policy. The upstream synchronization workflow is removed. The CI workflow comment retains the explanation that upstream workflows are disabled because they require self-hosted runners and secrets, but removes the reference to sync conflicts.

Priority: ⬇️ Low

Merge Risk: ⚪ Minimal · up to 6db2b

The repository will stop syncing upstream and document its security-only maintenance policy. No concrete user or release failure is identified in the supplied change context, so the PR is ready to merge after normal checks.

Pre-merge checks: Passed 5
Check name Status Explanation
Title check Passed The title clearly identifies the two main changes: freezing the fork and removing the weekly upstream sync workflow.
Description check Passed The description clearly states the purpose, lists the main file changes, documents the absence of crate, API, and wire changes, and records relevant notes. It does not use the template headings or inc…
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.
Full details: Description check

Explanation

The description clearly states the purpose, lists the main file changes, documents the absence of crate, API, and wire changes, and records relevant notes. It does not use the template headings or include the change checklist, but the core information is complete.

Full details: Docstring Coverage

Explanation

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 files. (2 skipped: 2 unsupported.)

✨ Finishing Touches
✨ Simplify code
  • Commit to this branch
  • Create a new PR
  • Autofix · 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.

kixelated added a commit to moq-dev/moq that referenced this pull request Oct 16, 2026
moq-dev/noq is frozen in moq-dev/noq#41, and every carried change in its
CHANGELOG-MOQ.md is ported or recorded as not applicable in #5664. Delete
the epic and drop its Required entries, rewording the prose that waited
on the switch.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@kixelated
kixelated merged commit 6af3f09 into main Oct 16, 2026
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