I missed this in my review. Oops! We should probably just be doing `return a.localeCompare(b);`