GitHub PushEvent at 2014-02-11 19:49:09
Repository: hr
Actor: Keith-S-Thompson
Payload: {"shas":[["945dca73d3d2691cc8bda9822fa9fa1d9bf95f19","c8cae0d3a54905ff83b55320f567b80e00935e8d@gmail.com","Changing the default space character.\n\nChanging the default space character to one that BASH will interpret as a comment.","gen0cide-",true],["fb0833ca0864cf978412cb9e2167f7695d0424f2","1a73af9e7ae00182733b2292511b814be66f065f@fengb.info","Eliminate a loop","fengb",true],["032149a8712f471fde216ae231d5f6e9d390e074","7d9da8c7031d4c135fbb677678a8e9b053e4444b@gmail.com","Update README.md\n\nmisspelling: ascci -> ascii","Matt Caldwell",true],["cc1349bdbe82ef81d16b5fa4754db360b66c3fc8","db65310e05190db7f4177804b0a95ebc87ac6a84@gmail.com","Merge pull request #12 from mattcaldwell/patch-1\n\nUpdate README.md","Gil Gonçalves",true],["2189915e406a9e3ad430a89db85ba9fd06a61d39","db65310e05190db7f4177804b0a95ebc87ac6a84@gmail.com","Merge pull request #8 from fengb/master\n\nImproved performance by eliminating a loop","Gil Gonçalves",true],["91811e2891ef6ead768add748cf1ab0d39499485","db65310e05190db7f4177804b0a95ebc87ac6a84@gmail.com","Merge branch 'master' of github.com:gen0cide-/hr into gen0cide--master\n\nConflicts:\n\thr","Gil Gonçalves",true],["53fca12d2f9b51ca9c66d5bf653e924dbe507183","db65310e05190db7f4177804b0a95ebc87ac6a84@gmail.com","Merge branch 'add-quotes' of github.com:Keith-S-Thompson/hr into Keith-S-Thompson-add-quotes\n\nConflicts:\n\thr","Gil Gonçalves",true],["103e25c7844cfed2dcf17fdcb366dbd6e45105ab","db65310e05190db7f4177804b0a95ebc87ac6a84@gmail.com","Merge branch 'multiple-arguments' of github.com:Keith-S-Thompson/hr into Keith-S-Thompson-multiple-arguments\n\nConflicts:\n\tREADME.md\n\thr","Gil Gonçalves",true],["682c75ec53296290d26dca699f39622612ff2fcb","5d838ec8c2c344259f3c8540e98ed35f35d69aac@markusfisch.de","More portable, sourceable and less dependencies\n\n#!/usr/bin/env bash is more portable than #!/bin/bash.\r\n\r\nUsed bash built-in shell variable COLUMNS instead of tput.\r\n\r\nUsed default values (${VAR:-N}) instead of if-then statement.\r\n\r\nMade the script sourceable so it can be used from other scripts.\r\n\r\nMade variables local.\r\n\r\nChanged environment variables to upper case. Every variable in a bash script is a environment variable which is \"global\" for all subsequent invocations; see:\r\n\r\nfoo()\r\n{\r\n    echo $FOOBAR\r\n}\r\n\r\nbar()\r\n{\r\n    local FOOBAR=7\r\n    foo\r\n}\r\n\r\nTo reflect that, a bash variable should be upper case. But YMMV, so it's up to you. Just wanted to explain why some people use upper case.\r\n\r\nMoved the ability to process multiple arguments in its own function. This way the tasks are atomized and the environment is not polluted by WORD because it can be local.","Markus Fisch",true],["bedb59eced972e77e848dc00948e581b10fc18ec","db65310e05190db7f4177804b0a95ebc87ac6a84@gmail.com","Merge pull request #15 from markusfisch/patch-1\n\nMore portable, sourceable and fewer dependencies","Gil Gonçalves",true],["6846ef3988279f9554421f49b579fffbdd02ecfe","db65310e05190db7f4177804b0a95ebc87ac6a84@gmail.com","Small code changes\n\n* Made open braces opening on the same declaration line\n* Re-added tput since the $COLUMNS variable was not set in my terminal\nand I may not be the only one.\n* Re-added an if for better readability","Gil Gonçalves",true]],"size":11,"ref":"refs/heads/multiple-arguments","head":"6846ef3988279f9554421f49b579fffbdd02ecfe"}