Skip to content

Flip remote code review flag#11537

Merged
MaggieShan merged 1 commit into
masterfrom
maggs/flip-code-review-ssh-ff
May 22, 2026
Merged

Flip remote code review flag#11537
MaggieShan merged 1 commit into
masterfrom
maggs/flip-code-review-ssh-ff

Conversation

@MaggieShan
Copy link
Copy Markdown
Contributor

Description

  • WISOTT
  • Launch for stable

@cla-bot cla-bot Bot added the cla-signed label May 22, 2026
@oz-for-oss
Copy link
Copy Markdown
Contributor

oz-for-oss Bot commented May 22, 2026

@MaggieShan

I'm starting a first review of this pull request.

You can view the conversation on Warp.

I completed the review and no human review was requested for this pull request.

Comment /oz-review on this pull request to retrigger a review (up to 3 times on the same pull request).

Powered by Oz

@MaggieShan MaggieShan requested a review from kevinyang372 May 22, 2026 14:31
Copy link
Copy Markdown
Contributor

@oz-for-oss oz-for-oss Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Overview

This PR promotes remote code review to Stable by enabling the remote_code_review Cargo feature by default and removing FeatureFlag::RemoteCodeReview from the Dogfood-only rollout list.

Concerns

  • This exposes a user-facing remote code review flow to Stable users, but the PR description does not include screenshots or a screen recording demonstrating the flow end to end.

Verdict

Found: 0 critical, 1 important, 0 suggestions

Request changes

Comment /oz-review on this pull request to retrigger a review (up to 3 times on the same pull request).

Powered by Oz

Comment thread app/Cargo.toml
@MaggieShan MaggieShan enabled auto-merge (squash) May 22, 2026 14:35
@MaggieShan MaggieShan merged commit e1070ff into master May 22, 2026
32 checks passed
@MaggieShan MaggieShan deleted the maggs/flip-code-review-ssh-ff branch May 22, 2026 14:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants