Skip to content

Remove moq-lite stub crate#1492

Merged
kixelated merged 1 commit into
mainfrom
claude/lucid-raman-66f07f
May 24, 2026
Merged

Remove moq-lite stub crate#1492
kixelated merged 1 commit into
mainfrom
claude/lucid-raman-66f07f

Conversation

@kixelated

Copy link
Copy Markdown
Collaborator

Summary

  • Drop rs/moq-lite/, the deprecated re-export of moq-net left over from the rename.
  • Remove its two entries from the workspace Cargo.toml and regenerate Cargo.lock.
  • Update the stale rs/moq-lite link in rs/conducer/README.md to point at rs/moq-net.
  • Tidy the migration callouts in rs/moq-net/README.md and rs/moq-net/CHANGELOG.md that linked to the now-deleted directory.

Notes for reviewers

  • No other crate in the workspace depends on moq-lite (confirmed via grep across all Cargo.toml files), so removal is a no-op for build graph consumers.
  • The 0.17.0 shim already published on crates.io remains as the final version for anyone still on the old name; nothing here unpublishes it.
  • Because the crate directory is gone, release-plz simply stops seeing it; no .release-plz.toml entry is needed (contrast with the moq-clock precedent in f9eb9db, which kept the crate around and set release = false).
  • Protocol-level references to moq-lite (the wire protocol, docs at /concept/layer/moq-lite, ALPN strings, etc.) are intentionally untouched.
  • just check passes locally.

Test plan

  • just check

🤖 Generated with Claude Code

The moq-lite crate was a deprecated re-export of moq-net kept around as
a migration shim after the rename. Drop the workspace member and update
the stale links that pointed at its source. The published 0.17.0 stays
on crates.io as the final shim for anyone still depending on the old
name.

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

coderabbitai Bot commented May 24, 2026

Copy link
Copy Markdown
Contributor

Warning

Review limit reached

@kixelated, we couldn't start this review because you've used your available PR reviews for now.

Your plan includes 4 reviews of capacity. Refill in 10 minutes and 26 seconds.

Your organization has run out of usage credits. Purchase more in the billing tab.

⌛ How to resolve this issue?

After more review capacity refills, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than trial, open-source, and free plans. In all cases, review capacity refills continuously over time.

Please see our FAQ for further information.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: de6847f5-4d0c-47a8-8c9b-df5cb932bad0

📥 Commits

Reviewing files that changed from the base of the PR and between a2086c6 and d4f94b6.

⛔ Files ignored due to path filters (1)
  • Cargo.lock is excluded by !**/*.lock
📒 Files selected for processing (9)
  • CLAUDE.md
  • Cargo.toml
  • rs/conducer/README.md
  • rs/moq-lite/CHANGELOG.md
  • rs/moq-lite/Cargo.toml
  • rs/moq-lite/README.md
  • rs/moq-lite/src/lib.rs
  • rs/moq-net/CHANGELOG.md
  • rs/moq-net/README.md
✨ Finishing Touches
✨ Simplify code
  • Create PR with simplified code
  • Commit simplified code in branch claude/lucid-raman-66f07f

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.

@kixelated kixelated enabled auto-merge (squash) May 24, 2026 22:43
@kixelated kixelated merged commit 42219bf into main May 24, 2026
1 check passed
@kixelated kixelated deleted the claude/lucid-raman-66f07f branch May 24, 2026 23:00
This was referenced May 24, 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