Skip to content

Instantly share code, notes, and snippets.

Avatar
🔐
[redacted]

Matt Todd mtodd

🔐
[redacted]
View GitHub Profile
@mtodd
mtodd / sample.geojson
Created Jul 26, 2017 — forked from landonreed/sample.geojson
Playing around with http://geojson.io #osm github etc.
View sample.geojson
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
View active_record_master_company_of_first_firm_test_failure.rb
begin
require 'bundler/inline'
rescue LoadError => e
$stderr.puts 'Bundler version 1.10 or later is required. Please update your Bundler'
raise e
end
gemfile(true) do
source 'https://rubygems.org'
gem 'rails', github: 'rails/rails'
View nested-search-pdus.rb
[#<Net::LDAP::PDU:0x007fa610a7bf58
@app_tag=4,
@ldap_controls=[],
@message_id=3,
@search_entry=
#<Net::LDAP::Entry:0x007fa610a7be90
@myhash=
{:dn=>["uid=user1,ou=People,dc=rubyldap,dc=com"],
:uid=>["user1"],
:cn=>["user1"],
View Bug.md

[ ] a [ ] b

  • c
  • d

If user is careless, forgetting to put "- " before "[ ]", TODO list could get buggy.

Steps to reproduce bug:

  • on parsed MD page of list above, tick "c" to note that "c" is done.
  • Now, edit the file.
View keybase.md

Keybase proof

I hereby claim:

  • I am mtodd on github.
  • I am mtodd (https://keybase.io/mtodd) on keybase.
  • I have a public key whose fingerprint is C5BB 8DA1 86D5 1E58 CFF0 DE41 CBAD C21E 22F3 20F3

To claim this, I am signing this object:

View Broken.md
  • First option
  • Parent option
  • Nested option
  • Last option
View code_review_checklists.md

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

@mtodd
mtodd / _.md
Created Apr 22, 2013
Tributary inlet
@mtodd
mtodd / _.md
Created Apr 19, 2013
sin waves
View _.md
@mtodd
mtodd / jsonpp.rb
Created Nov 13, 2011
PrettyPrint JSON
View jsonpp.rb
require 'rubygems'
require 'yajl/json_gem' # or 'json'
require 'yaml'
puts JSON.parse($stdin.read.chomp).to_yaml