lix-project/meta

gerrit cli #18

New issue

Open

opened 2024-03-26 22:26:02 +00:00 by jade · 3 comments

jade commented

2024-03-26 22:26:02 +00:00

Owner

@rbt requested

there aren't good gerrit clis at the moment. this is very silly and we really ought to have something nicer.

@rbt requested there aren't good gerrit clis at the moment. this is very silly and we really ought to have something nicer.

rbt commented

2024-03-26 22:55:17 +00:00

Owner

Features I'd like:

gerrit push instead of typing out the magic git push origin HEAD:refs/for/main invocation
gerrit checkout 452 to checkout a CL
gerrit down to check out the previous CL in a stack (and gerrit up for the next one)
Something that exports review comments to a Vim-compatible :cfile

Features I'd like: - `gerrit push` instead of typing out the magic `git push origin HEAD:refs/for/main` invocation - `gerrit checkout 452` to checkout a CL - `gerrit down` to check out the previous CL in a stack (and `gerrit up` for the next one) - Something that exports review comments to [a Vim-compatible `:cfile`](https://vimdoc.sourceforge.net/htmldoc/quickfix.html#:cfile)

pennae commented

2024-03-26 22:58:34 +00:00

Owner

for reference a few aliases we use:

[alias]
    fcl = !git fetch origin refs/changes/"${1:$((${#1}-2)):2}"/"$1"/"${2:-1}" && true
	gpush = push origin HEAD:refs/for/main
	# cocl = !git fcl "$@" && git checkout FETCH_HEAD && true
	cocl = !\
	 ps=$(\
	   ssh $(git config remote.origin.gerriturl) \
	     gerrit query --format=json --current-patch-set $1 \
        | jq -sr .[0].currentPatchSet.ref \
	 ) && git fetch origin $ps && git checkout FETCH_HEAD && true

that does "push" (gpush) and "checkout" (cocl). "down"/"up" should be easy enough to add by asking the current commit for its change-id, asking gerrit for the prev/next change, and checking that out?

for reference a few aliases we use: ``` [alias] fcl = !git fetch origin refs/changes/"${1:$((${#1}-2)):2}"/"$1"/"${2:-1}" && true gpush = push origin HEAD:refs/for/main # cocl = !git fcl "$@" && git checkout FETCH_HEAD && true cocl = !\ ps=$(\ ssh $(git config remote.origin.gerriturl) \ gerrit query --format=json --current-patch-set $1 \ | jq -sr .[0].currentPatchSet.ref \ ) && git fetch origin $ps && git checkout FETCH_HEAD && true ``` that does "push" (`gpush`) and "checkout" (`cocl`). "down"/"up" should be easy enough to add by asking the current commit for its change-id, asking gerrit for the prev/next change, and checking that out?

irenes commented

2024-03-27 01:59:36 +00:00

Owner

as a reminder also, I documented a way to avoid the need for the push magic at https://wiki.lix.systems/books/lix-contributors/page/gerrit#bkmrk-pushing

it's particularly nice in that it's just a one-time thing you do at the time you check the repo out, much like installing the commit-msg hook

as a reminder also, I documented a way to avoid the need for the push magic at https://wiki.lix.systems/books/lix-contributors/page/gerrit#bkmrk-pushing it's particularly nice in that it's just a one-time thing you do at the time you check the repo out, much like installing the commit-msg hook