allowing yadm to use the neogit ui
Nobody has claimed this yet.
- Dominant language
- Lua
- Stars
- 5.6k
- Forks
- 357
- Avg merge
- 15h 47m
- Merged PRs (30d)
- 2
Description
Is your feature request related to a problem? Please describe.
I enjoy this library and within emacs I was able to use yadm with magit through a simple script.
Here However it seemed a bit harder to script easily with the current structure.
Describe the solution you'd like
It would be nice to allow the user to choose to use the neogit gui for yadm through a simple command suchj as neogitYadm
Describe alternatives you've considered
I tried to script it in a similar way as this which works in emacs magit but due to structural differences this was not possible:
(use-package tramp
:config
(add-to-list 'tramp-methods
'("yadm"
(tramp-login-program "yadm")
(tramp-login-args (("enter")))
(tramp-login-env (("SHELL") ("/bin/sh")))
(tramp-remote-shell "/bin/sh")
(tramp-remote-shell-args ("-c")))))
(defun magit-yadm ()
(interactive)
(magit-status "/yadm::")t
)
Additional context
I have created my own fork with a solution that works for me. It is a bit of a dirty fix at the moment though and probably does not adhere to your code standards as of yet. If other people are interested I'd be happy to clean it up for wider use and create a merge request 😄
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by reviewing Neogit's existing command and UI entry points, then compare the linked fork's yadm integration with the current structure. Done means users can invoke the Neogit UI for yadm through a simple command without relying on a private script.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- git, lua, neovim
- Domain
- cli, devtools
- Issue type
- Feature
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 30/100