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
560790043	NR: 2.10 Arabic Music - inlcude references to hel-arabic.ly (Closed)	lilypond-pkx	anthoneyhughes88, hassan.elfatihi, lemzwerg	20	1 day, 12 hours
576830043	NR: 2.10 Arabic Music - inlcude references to hel-arabic.ly (Closed)	lilypond-pkx			1 week, 6 days
552770043	Add glib-2.0 and gobject-2.0 library dependency (Closed)	lilypond-pkx	lemzwerg	10	4 weeks, 1 day
562850043	Add ly version t0 swing.ly (Closed)	thomasmorley651			1 month
231680043	Issue 4365: non-member unsmob<T> etc. (Closed)	Dan Eble	dak, dan	8	1 month, 1 week
2627043	code review 2627043: go spec: append built-in (Closed)	gri	r, rsc1	7	1 month, 2 weeks
560670043	Document bound-details (sub-)properties in line-spanner-cc for IR (Closed)	thomasmorley651	dak, lemzwerg	21	1 month, 3 weeks
556730043	Sends the agent recorded list bytes in the pulse message. (Closed)	alanmorales	akaiser	42	2 months
558740043	Adds the agent uptime field to the agent pulse. (Closed)	alanmorales	akaiser	3	2 months
550650043	Documentation: elaborate NR `custom percussion’ section (Closed)	Valentin Villenave	lemzwerg	3	2 months
566610043	Adds missing package.json, adds --save option to npm install to ensure component is saved to node_m… (Closed)	alanmorales	mkmg		2 months, 1 week
560620043	Reflect new syntax in example of define-grob-properties.scm (Closed)	thomasmorley651			2 months, 3 weeks
556640043	Add doc-string for allowVoltaHook (Closed)	thomasmorley651			2 months, 3 weeks
582530043	cldrbug 11951: Add cldrVersion key to locale_dependencies JSON file (Closed)	sffc	pedberg		3 months, 4 weeks
572520044	Fix #687: Include MIDI swing script in default distribution (Closed)	Valentin Villenave	dak, thomasmorley651	15	3 months, 4 weeks
5646055	code review 5646055: cmd/dist: make 'make' quieter for developers (Closed)	rsc	innnzzz6, gri		4 months
343460043	Add glog log_dir creation to agentmain (Closed)	thobrla	akaiser	2	4 months, 1 week
348120043	Allow scripts to be defined either as glyphs or stencils. (Closed)	Valentin Villenave	dak, rap.ardiputra127	22	4 months, 2 weeks
353880043	C++ cleanup : get rid of some compilation warnings (Closed)	Valentin Villenave	james.lowe, dak, lemzwerg, John Mandereau	4	4 months, 2 weeks
343510043	Mention FILE or FOLDER arg to -o command-line option. (Closed)	Valentin Villenave			4 months, 2 weeks

1 - 20 Older ›