feat(magit): replace github-review for code-review

This commit is contained in:
Ellis Kenyo 2021-12-15 07:05:57 +00:00
parent 23f6f90515
commit 2d3a68df49
No known key found for this signature in database
GPG key ID: 298BE5D997EBAA02
3 changed files with 17 additions and 7 deletions

View file

@ -153,9 +153,9 @@ kill all magit buffers for this repo."
(kill-buffer buf)))))))
;;;###autoload
(defun +magit/start-github-review (arg)
(defun +magit/start-code-review (arg)
(interactive "P")
(call-interactively
(if (or arg (not (featurep 'forge)))
#'github-review-start
#'github-review-forge-pr-at-point)))
#'code-review-start
#'code-review-forge-pr-at-point)))

View file

@ -188,14 +188,23 @@ ensure it is built when we actually use Forge."
(add-hook hook #'forge-bug-reference-setup)))))))
(use-package! github-review
(use-package! code-review
:after magit
:init
;; TODO This needs to either a) be cleaned up or better b) better map things
;; to fit
(after! evil-collection-magit
(dolist (binding evil-collection-magit-mode-map-bindings)
(pcase-let* ((`(,states _ ,evil-binding ,fn) binding))
(dolist (state states)
(evil-collection-define-key state 'code-review-mode-map evil-binding fn))))
(evil-set-initial-state 'code-review-mode evil-default-state))
:config
(transient-append-suffix 'magit-merge "i"
'("y" "Review pull request" +magit/start-github-review))
'("y" "Review pull request" +magit/start-code-review))
(after! forge
(transient-append-suffix 'forge-dispatch "c u"
'("c r" "Review pull request" +magit/start-github-review))))
'("c r" "Review pull request" +magit/start-code-review))))
(use-package! magit-todos

View file

@ -6,4 +6,5 @@
(package! forge :pin "41efa674cff0b447efbc103494fd61ec9b9156ae"))
(package! magit-gitflow :pin "cc41b561ec6eea947fe9a176349fb4f771ed865b")
(package! magit-todos :pin "60152d5c4e4b73e72e15f23ca16e8cc7734906bc")
(package! github-review :pin "725fbc7b385228f53a7ddc46a92c1276bab4aea8"))
(package! code-review :pin "f1a79c20ae51d23f76067a1e5a2f5c1c4db42ec9"
:recipe (:files ("graphql" "code-review*.el"))))