Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Move git status out of Entry #22224

Draft
wants to merge 21 commits into
base: main
Choose a base branch
from
Draft

Move git status out of Entry #22224

wants to merge 21 commits into from

Conversation

mikayla-maki
Copy link
Contributor

@mikayla-maki mikayla-maki commented Dec 19, 2024

  • Rewrite worktree git handling
  • Fix tests
  • Fix test_propagate_statuses_for_repos_under_project
  • Replace WorkDirectoryEntry with WorkDirectory in RepositoryEntry
  • Add a worktree event for capturing git status changes
  • Confirm that the local repositories are correctly updating the new WorkDirectory field
  • Implement the git statuses query as a join when pulling entries out of worktree
  • Use this new join to implement the project panel and outline panel.
  • Synchronize git statuses over the wire for collab and remote dev

Release Notes:

  • N/A

cole-miller and others added 4 commits December 16, 2024 09:51
the git panel

co-authored-by: Mikayla <mikayla@zed.com>
mark places that used to use it for refactoring

co-authored-by: Cole <cole@zed.dev>
Co-authored-by: Mikayla <mikayla@zed.dev>
Co-authored-by: Nathan <nathan@zed.dev>
Copy link

cla-bot bot commented Dec 19, 2024

Thank you for your pull request and welcome to our community. We could not parse the GitHub identity of the following contributors: Cole Miller.
This is most likely caused by a git client misconfiguration; please make sure to:

  1. check if your git client is configured with an email to sign commits git config --list | grep email
  2. If not, set it up using git config --global user.email email@example.com
  3. Make sure that the git commit email is configured in your GitHub account settings, see https://github.com/settings/emails

@cla-bot cla-bot bot added the cla-signed The user has signed the Contributor License Agreement label Dec 19, 2024
cole-miller and others added 9 commits December 19, 2024 17:23
co-authored-by: Cole <cole@zed.dev>
co-authored-by: cole <cole@zed.dev>
Co-authored-by: Mikayla <mikayla@zed.dev>
to the local and repository entries, and enhance the RepositoryWorkDirectory type to
make the worktree code more ergonomic.
APIs feel good, concept feels good.... now we just need to make it work.
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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants