Skip to content

git: Remove unused actions#49901

Merged
dinocosta merged 1 commit intomainfrom
49288-remove-unused-actions
Feb 23, 2026
Merged

git: Remove unused actions#49901
dinocosta merged 1 commit intomainfrom
49288-remove-unused-actions

Conversation

@dinocosta
Copy link
Member

@dinocosta dinocosta commented Feb 23, 2026

Remove both git::LoadMoreHistory and git::ViewCommitFromHistory actions as although both actions were registered in the workspace, they don't actually have effect when run.

This appears to have been leftover from
#42441 , possibly the initial implementation on how to open a commit and load more commits when the file history view was open, which is now handled with Button.on_click callbacks instead of relying on actions.

Closes #49288

Before you mark this PR as ready for review, make sure that you have:

  • Added a solid test coverage and/or screenshots from doing manual testing
  • Done a self-review taking into account security and performance aspects
  • Aligned any UI changes with the UI checklist

Release Notes:

  • Removed unused git actions – git: load more history and git: view commit from history

Remove both `git::LoadMoreHistory` and `git::ViewCommitFromHistory`
actions as although both actions were registered in the workspace, they
don't actually have effect when run.

This appears to have been leftover from
#42441 , possibly the initial
implementation on how to open a commit and load more commits when the
file history view was open, which is now handled with `Button.on_click`
callbacks instead of relying on actions.
@cla-bot cla-bot bot added the cla-signed The user has signed the Contributor License Agreement label Feb 23, 2026
@zed-community-bot zed-community-bot bot added the staff Pull requests authored by a current member of Zed staff label Feb 23, 2026
@dinocosta dinocosta self-assigned this Feb 23, 2026
@dinocosta dinocosta changed the title refactor: remove unused git actions git: Remove unused actions Feb 23, 2026
@dinocosta dinocosta marked this pull request as ready for review February 23, 2026 15:10
@dinocosta dinocosta merged commit 61757aa into main Feb 23, 2026
41 checks passed
@dinocosta dinocosta deleted the 49288-remove-unused-actions branch February 23, 2026 15:23
@Dima-369 Dima-369 mentioned this pull request Feb 23, 2026
3 tasks
Anthony-Eid pushed a commit to bobbymannino/zed that referenced this pull request Feb 25, 2026
Remove both `git::LoadMoreHistory` and `git::ViewCommitFromHistory`
actions as although both actions were registered in the workspace, they
don't actually have effect when run.

This appears to have been leftover from
zed-industries#42441 , possibly the initial
implementation on how to open a commit and load more commits when the
file history view was open, which is now handled with `Button.on_click`
callbacks instead of relying on actions.

Closes zed-industries#49288 

Release Notes:

- Removed unused git actions – `git: load more history` and `git: view
commit from history`
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.

Git: Unused actions "git: view commit from history" and "git: load more history"

1 participant