alphapapa / alphapapa/magit-todos

Idea: use (magit-get-upstream-ref) as ref commit

Ouverte
#155 10 commentaires 0 réactions 0 personnes assignées Voir sur GitHub
enhancement good first issue help wanted
Langage dominant
Emacs Lisp
Étoiles
791
Forks
54
Métriques de merge des PR
Aucune PR mergée en 30 j

Description

This is just an idea/suggestion. I see that magit-todos uses `(magit-main-branch)` to figure out (using `merge-base`) the revision to compare the HEAD against.

How about using `(magit-get-upstream-ref)` instead? I'm suggesting this, because people might not update their `main` branch for a long time, so it gets out of sync, and `merge-base` may find some very ancient revision, so we get a lot of entries in the TODO list.
On the other hand, `(magit-get-upstream-ref)` is usually more up-to-date, and it also more likely contains a more relevant revision to find the merge base.

What do you think?

I've been using this idea for a while, and it seems to work for me. If anyone interested, I use this snippet to achieve this:

```lisp
(defun my-magit-mode-hook ()
(let ((upstream (magit-get-upstream-ref)))
(when upstream
(setq magit-todos-branch-list-merge-base-ref upstream)))
(add-hook 'magit-mode-hook 'my-magit-mode-hook)
```

Guide de contribution

Aucun guide de contribution indexé pour ce dépôt

Évaluation

Cette issue n'a pas encore été évaluée.

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.