-
Notifications
You must be signed in to change notification settings - Fork 5
Comparison of gerrit.el with egerrit.el
Thomas Hisch edited this page Aug 18, 2022
·
4 revisions
| Feature | egerrit.el | gerrit.el |
|---|---|---|
| http requests | curl subprocess | url.el |
| git calls | git subprocess | magit |
| support for commenting on changes | ✅ | ❌ |
| faces | new ones are defined | magit faces are re-used |
| command selection | ?? | transient.el menu for uploading/downloading changes |
| CI results | ✅ (zuul via zuul.el) | ❌ |
TODO add more stuff
- gerrit-rest.el should use a function similiar to
egerrit-request-url, which allows passing the parms of the request url as an alist - Consider using the relatively new plz.el (part to elpa.git) for performing http requests
- Consider using a library (preferably in elpa.git) that performs git subprocess/libgit2 calls instead of using magit (gerrit.el) or re-inventing the wheel (egerrit.el).