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
556730043	Sends the agent recorded list bytes in the pulse message. (Closed)	alanmorales	akaiser	42	3 days, 16 hours
558740043	Adds the agent uptime field to the agent pulse. (Closed)	alanmorales	akaiser	3	4 days, 15 hours
550650043	Documentation: elaborate NR `custom percussion’ section (Closed)	Valentin Villenave	lemzwerg	3	5 days, 3 hours
566610043	Adds missing package.json, adds --save option to npm install to ensure component is saved to node_m… (Closed)	alanmorales	mkmg		1 week, 4 days
560620043	Reflect new syntax in example of define-grob-properties.scm (Closed)	thomasmorley651			4 weeks
556640043	Add doc-string for allowVoltaHook (Closed)	thomasmorley651			4 weeks, 1 day
582530043	cldrbug 11951: Add cldrVersion key to locale_dependencies JSON file (Closed)	sffc	pedberg		2 months
572520044	Fix #687: Include MIDI swing script in default distribution (Closed)	Valentin Villenave	dak, thomasmorley651	15	2 months
5646055	code review 5646055: cmd/dist: make 'make' quieter for developers (Closed)	rsc	innnzzz6, gri		2 months, 1 week
343460043	Add glog log_dir creation to agentmain (Closed)	thobrla	akaiser	2	2 months, 1 week
348120043	Allow scripts to be defined either as glyphs or stencils. (Closed)	Valentin Villenave	dak, rap.ardiputra127	22	2 months, 2 weeks
353880043	C++ cleanup : get rid of some compilation warnings (Closed)	Valentin Villenave	james.lowe, dak, lemzwerg, John Mandereau	4	2 months, 3 weeks
343510043	Mention FILE or FOLDER arg to -o command-line option. (Closed)	Valentin Villenave			2 months, 3 weeks
363910043	Use "simple strings" rather than #"hash-prefixed Scheme strings" (Closed)	Valentin Villenave	dak, thomasmorley651, carl.d.sorensen	3	2 months, 3 weeks
225380043	C++/SCM typing strictness, round 2 (Closed)	Valentin Villenave	pkx166h, dak, Dan Eble		2 months, 3 weeks
225040043	Embed LilyPond source files inside generated PDF (Closed)	Valentin Villenave	dak	23	2 months, 3 weeks
288290043	Define French as a separate input language (Closed)	Valentin Villenave	pacovila, dak	8	2 months, 3 weeks
221710044	note-name-engraver: add user-defined note names support (Closed)	Valentin Villenave	Dan Eble, dak	14	2 months, 3 weeks
367900043	Mention hacklily.org on the website. (Closed)	Valentin Villenave			2 months, 3 weeks
342340043	Add news item for ukulele changes by Davide, maybe? (Closed)	Valentin Villenave			2 months, 3 weeks

1 - 20 Older ›