Skip to content

Conversation

@techknowlogick
Copy link
Member

following the approach from nixpkgs that ignore the .worktrees folder, we could also do the same, this would allow worktrees to be worked on in the same folder as the primary branch.

ref: NixOS/nixpkgs@b6420c7

following the approach from nixpkgs that ignore the .worktrees folder,
we could also do the same, this would allow worktrees to be worked on in
the same folder as the primary branch.

ref: NixOS/nixpkgs@b6420c7
@techknowlogick techknowlogick added the skip-changelog This PR is irrelevant for the (next) changelog, for example bug fixes for unreleased features. label Nov 3, 2025
@GiteaBot GiteaBot added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label Nov 3, 2025
@silverwind
Copy link
Member

silverwind commented Nov 3, 2025

Is it .worktree (like in nxpkgs) or .worktrees?

@techknowlogick
Copy link
Member Author

@silverwind I added the s to make it plural, so it doesn't exactly follow their naming, but it follows the approach of having some folder in the repo root that contains all worktrees.

Copy link
Member

@silverwind silverwind left a comment

Choose a reason for hiding this comment

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

I don't know what a worktree is but fine in any case.

@GiteaBot GiteaBot added lgtm/need 1 This PR needs approval from one additional maintainer to be merged. and removed lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. labels Nov 3, 2025
@GiteaBot GiteaBot added lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. and removed lgtm/need 1 This PR needs approval from one additional maintainer to be merged. labels Nov 3, 2025
@techknowlogick techknowlogick merged commit aa7ec64 into go-gitea:main Nov 3, 2025
25 checks passed
@techknowlogick techknowlogick deleted the add-worktrees-to-gitignore branch November 3, 2025 19:57
@GiteaBot GiteaBot added this to the 1.26.0 milestone Nov 3, 2025
@techknowlogick
Copy link
Member Author

I don't know what a worktree is but fine in any case.

@silverwind I find them helpful as they lessen the copies of the .git directory I have when working on multiple branches simultaneously (basically having multiple folders share the same .git folder objects so they don't need to be duplicated. So say I could do a git fetch origin, and then each "copy" would then have access to all of the new objects so I could then git merge origin/main to keep things up to date).

zjjhot added a commit to zjjhot/gitea that referenced this pull request Nov 6, 2025
* giteaofficial/main:
  Remove padding override on `.ui .sha.label` (go-gitea#35864)
  fix(api/repo/contents): set the dates to now when not specified by the caller (go-gitea#35861)
  Remove `fix` Make targets (go-gitea#35868)
  Refactor ls-tree and git path related problems (go-gitea#35858)
  Fix pull description code label background (go-gitea#35865)
  Make ACME email optional (go-gitea#35849)
  Remove wrong code (go-gitea#35846)
  Fix Arch repo pacman.conf snippet (go-gitea#35825)
  Port away from `flake-utils` (go-gitea#35675)
  Update golangci-lint to v2.6.0 (go-gitea#35801)
  Add a doctor command to fix inconsistent run status (go-gitea#35840)
  Fix viewed files number is not right if not all files loaded (go-gitea#35821)
  Fix incorrect pull request counter (go-gitea#35819)
  Fix a number of `strictNullChecks`-related issues (go-gitea#35795)
  ignore .worktrees as a "special folder" (go-gitea#35835)
  upgrade go mail to 0.7.2 and fix the bug (go-gitea#35833)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. skip-changelog This PR is irrelevant for the (next) changelog, for example bug fixes for unreleased features.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants