Code Review

Keyboard Shortcuts

	File
u :	up to issue
m :	publish + mail comments
M :	edit review message
j / k :	jump to file after / before current file
J / K :	jump to next file with a comment after / before current file
	Side-by-side diff
i :	toggle intra-line diffs
e :	expand all comments
c :	collapse all comments
s :	toggle showing all comments
n / p :	next / previous diff chunk or comment
N / P :	next / previous comment
<Up> / <Down> :	next / previous line
<Enter> :	respond to / edit current comment
d :	mark current comment as done

	Issue
u :	up to list of issues
m :	publish + mail comments
j / k :	jump to patch after / before current patch
o / <Enter> :	open current patch in side-by-side view
i :	open current patch in unified diff view

	Issue List
j / k :	jump to issue after / before current issue
o / <Enter> :	open current issue
# :	close issue

	Comment/message editing
<Ctrl> + s or <Ctrl> + Enter :	save comment
<Esc> :	cancel edit

Recent Closed Issues

1 - 20 Older ›

	Subject	Owner	Reviewers	Comments	Last updated
558820043	flask_tryton: Add support for 5.2 series (Closed)	pokoli	resteve, ced	2	1 month
574940043	Updates the changelog for version v2.0.2 release. (Closed)	alanmorales	thobrla		1 month
6465078	Script to sanitize source files (Closed)	rmistry	epoger, EricB, bsalomon, status.rpni	9	1 month, 1 week
6733055	code review 6733055: cmd/5g, cmd/6g: fix out of registers with array indexing. (Closed)	remyoudompheng	rsc, Nick name	4	1 month, 2 weeks
287970043	Restrict transforms to GSA, fully trusted clients (Closed)	myk	baby546833, pjo	28	2 months
560790043	NR: 2.10 Arabic Music - inlcude references to hel-arabic.ly (Closed)	lilypond-pkx	anthoneyhughes88, hassan.elfatihi, lemzwerg	20	2 months
576830043	NR: 2.10 Arabic Music - inlcude references to hel-arabic.ly (Closed)	lilypond-pkx			2 months, 2 weeks
552770043	Add glib-2.0 and gobject-2.0 library dependency (Closed)	lilypond-pkx	lemzwerg	10	3 months
562850043	Add ly version t0 swing.ly (Closed)	thomasmorley651			3 months, 1 week
231680043	Issue 4365: non-member unsmob<T> etc. (Closed)	Dan Eble	dak, dan	8	3 months, 2 weeks
2627043	code review 2627043: go spec: append built-in (Closed)	gri	r, rsc1	7	3 months, 3 weeks
560670043	Document bound-details (sub-)properties in line-spanner-cc for IR (Closed)	thomasmorley651	dak, lemzwerg	21	3 months, 4 weeks
556730043	Sends the agent recorded list bytes in the pulse message. (Closed)	alanmorales	akaiser	42	4 months
558740043	Adds the agent uptime field to the agent pulse. (Closed)	alanmorales	akaiser	3	4 months, 1 week
550650043	Documentation: elaborate NR `custom percussion’ section (Closed)	Valentin Villenave	lemzwerg	3	4 months, 1 week
566610043	Adds missing package.json, adds --save option to npm install to ensure component is saved to node_m… (Closed)	alanmorales	mkmg		4 months, 2 weeks
560620043	Reflect new syntax in example of define-grob-properties.scm (Closed)	thomasmorley651			5 months
556640043	Add doc-string for allowVoltaHook (Closed)	thomasmorley651			5 months
582530043	cldrbug 11951: Add cldrVersion key to locale_dependencies JSON file (Closed)	sffc	pedberg		6 months
572520044	Fix #687: Include MIDI swing script in default distribution (Closed)	Valentin Villenave	dak, thomasmorley651	15	6 months

1 - 20 Older ›