Skip to content

Enable handoff feature flag by default for staff - #58891

Merged
bennetbo merged 1 commit into
mainfrom
AI-381/enable-handoff-for-staff
Jun 9, 2026
Merged

Enable handoff feature flag by default for staff#58891
bennetbo merged 1 commit into
mainfrom
AI-381/enable-handoff-for-staff

Conversation

@rtfeldman

Copy link
Copy Markdown
Contributor

Enables the "handoff" feature flag by default for Zed staff by removing the enabled_for_staff() override on HandoffFeatureFlag, which falls back to the trait default of true.

Closes AI-381

Release Notes:

  • N/A

@rtfeldman rtfeldman self-assigned this Jun 9, 2026
@cla-bot cla-bot Bot added the cla-signed The user has signed the Contributor License Agreement label Jun 9, 2026
@zed-community-bot zed-community-bot Bot added the staff Pull requests authored by a current member of Zed staff label Jun 9, 2026
@bennetbo
bennetbo marked this pull request as ready for review June 9, 2026 11:39
@bennetbo
bennetbo added this pull request to the merge queue Jun 9, 2026
Merged via the queue into main with commit ee5a1b4 Jun 9, 2026
43 checks passed
@bennetbo
bennetbo deleted the AI-381/enable-handoff-for-staff branch June 9, 2026 11:42
@zed-zippy zed-zippy Bot added the PR state:needs review Used to label PRs that are in need of a post-merge approval label Jun 9, 2026
@bennetbo

bennetbo commented Jun 9, 2026

Copy link
Copy Markdown
Member

@zed-industries/approved

@zed-zippy zed-zippy Bot removed the PR state:needs review Used to label PRs that are in need of a post-merge approval label Jun 9, 2026
pull Bot pushed a commit to relaxcn/zed that referenced this pull request Jun 9, 2026
Accidentally broke CI in zed-industries#58891

Since
https://github.com/zed-industries/zed/blob/c68bc98bacb80c34df0e1b6ef4de186d763b3820/crates/feature_flags/src/store.rs#L180
defaults to true in debug mode (effectively sets staff to `true`) the
tests would fail.

We can work around this by manually disabling the feature flag for tests
in settings.

Also, since we are planning to ship to preview tomorrow these tests will
be removed soon.


Release Notes:

- N/A
This was referenced Jun 18, 2026
jonx pushed a commit to jonx/zed-aros that referenced this pull request Jul 17, 2026
Enables the "handoff" feature flag by default for Zed staff by removing
the `enabled_for_staff()` override on `HandoffFeatureFlag`, which falls
back to the trait default of `true`.

Closes AI-381

Release Notes:

- N/A
jonx pushed a commit to jonx/zed-aros that referenced this pull request Jul 17, 2026
Accidentally broke CI in zed-industries#58891

Since
https://github.com/zed-industries/zed/blob/c68bc98bacb80c34df0e1b6ef4de186d763b3820/crates/feature_flags/src/store.rs#L180
defaults to true in debug mode (effectively sets staff to `true`) the
tests would fail.

We can work around this by manually disabling the feature flag for tests
in settings.

Also, since we are planning to ship to preview tomorrow these tests will
be removed soon.


Release Notes:

- N/A
jolutz pushed a commit to jolutz/zed that referenced this pull request Aug 8, 2026
Enables the "handoff" feature flag by default for Zed staff by removing
the `enabled_for_staff()` override on `HandoffFeatureFlag`, which falls
back to the trait default of `true`.

Closes AI-381

Release Notes:

- N/A
jolutz pushed a commit to jolutz/zed that referenced this pull request Aug 8, 2026
Accidentally broke CI in zed-industries#58891

Since
https://github.com/zed-industries/zed/blob/c68bc98bacb80c34df0e1b6ef4de186d763b3820/crates/feature_flags/src/store.rs#L180
defaults to true in debug mode (effectively sets staff to `true`) the
tests would fail.

We can work around this by manually disabling the feature flag for tests
in settings.

Also, since we are planning to ship to preview tomorrow these tests will
be removed soon.


Release Notes:

- N/A
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla-signed The user has signed the Contributor License Agreement staff Pull requests authored by a current member of Zed staff

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants