Gabriel Vanca gabriel-vanca

Based on the article: Using checklists for code review

In general, people are pretty good at the code review process, but it's sometimes surprising what can slip through. A natural consequence of the way our brains look at the world is that it's easy to pay a lot of attention to small details and code style flubs, and completely miss the big picture.

Obviously, not everything is applicable for every change. If the review request isn't making any changes to UI, then skip the first two checklists entirely. If a change is a bug fix, typically don't review it for architecture and design principles.

Put the big stuff first (e.g. architecture). You don't want to work through a ton of small issues before realizing that everything has to be rewritten.

Do a pass through the code for each and every item in the checklist. By only looking for a very specific type of defect, each pass goes relatively quickly, even for large changes. Focu

tmux shortcuts & cheatsheet

start new:

tmux

start new with session name:

tmux new -s myname

	function Get-CommandSource {
	param (
	[Parameter(Mandatory)]
	[String]$Name
	)

	try {
	$commandInfo = Get-Command $Name
	if ($commandInfo -is [System.Management.Automation.AliasInfo]) {
	$commandInfo = $commandInfo.ResolvedCommand

	<#
	PowerShell XAML Template
	by QuietusPlus
	#>

	<#
	Include
	#>

	# .NET Framework classes

	class A
	{
	public:
	int x;
	protected:
	int y;
	private:
	int z;
	};