{"action":"opened","number":37,"pull_request":{"state":"open","review_comments":0,"created_at":"2011-11-29T01:52:33Z","patch_url":"https://github.com/Jakobo/inject/pull/37.patch","changed_files":9,"updated_at":"2011-11-29T01:52:33Z","_links":{"review_comments":{"href":"https://api.github.com/repos/Jakobo/inject/pulls/37/comments"},"comments":{"href":"https://api.github.com/repos/Jakobo/inject/issues/37/comments"},"html":{"href":"https://github.com/Jakobo/inject/pull/37"},"self":{"href":"https://api.github.com/repos/Jakobo/inject/pulls/37"}},"user":{"gravatar_id":"e7df5aa388c83e07e3f0dcd582a9a6c1","id":1795,"url":"https://api.github.com/users/Jakobo","avatar_url":"https://secure.gravatar.com/avatar/e7df5aa388c83e07e3f0dcd582a9a6c1?d=https://a248.e.akamai.net/assets.github.com%2Fimages%2Fgravatars%2Fgravatar-140.png","login":"Jakobo"},"issue_url":"https://github.com/Jakobo/inject/issues/37","comments":0,"deletions":70,"body":"Hey all, here's my first attempt at the addRule code. I put it into a separate branch to collect feedback. The addRule example seems to be working, but would love to get some overall commentary on the approach. I feel kind of like in doing this I barfed all over onModuleLoad by adding two nested functions. Maybe execRan() can be pulled out for clarity?","merged":false,"commits":1,"mergeable":null,"id":536185,"base":{"user":{"gravatar_id":"e7df5aa388c83e07e3f0dcd582a9a6c1","id":1795,"url":"https://api.github.com/users/Jakobo","avatar_url":"https://secure.gravatar.com/avatar/e7df5aa388c83e07e3f0dcd582a9a6c1?d=https://a248.e.akamai.net/assets.github.com%2Fimages%2Fgravatars%2Fgravatar-140.png","login":"Jakobo"},"repo":{"created_at":"2011-09-08T00:38:32Z","pushed_at":"2011-11-29T01:50:23Z","updated_at":"2011-11-29T01:50:24Z","private":false,"clone_url":"https://github.com/Jakobo/inject.git","name":"inject","master_branch":null,"watchers":10,"svn_url":"https://svn.github.com/Jakobo/inject","language":"JavaScript","forks":6,"owner":{"gravatar_id":"e7df5aa388c83e07e3f0dcd582a9a6c1","id":1795,"url":"https://api.github.com/users/Jakobo","avatar_url":"https://secure.gravatar.com/avatar/e7df5aa388c83e07e3f0dcd582a9a6c1?d=https://a248.e.akamai.net/assets.github.com%2Fimages%2Fgravatars%2Fgravatar-140.png","login":"Jakobo"},"id":2345470,"homepage":"","git_url":"git://github.com/Jakobo/inject.git","description":"Dependency Management got Awesome","html_url":"https://github.com/Jakobo/inject","size":284,"open_issues":10,"fork":false,"url":"https://api.github.com/repos/Jakobo/inject","ssh_url":"git@github.com:Jakobo/inject.git"},"sha":"e6ba391c9ea1f8bec1c54374698d5138dcb990ad","ref":"master","label":"Jakobo:master"},"additions":294,"number":37,"merged_by":null,"head":{"user":{"gravatar_id":"e7df5aa388c83e07e3f0dcd582a9a6c1","id":1795,"url":"https://api.github.com/users/Jakobo","avatar_url":"https://secure.gravatar.com/avatar/e7df5aa388c83e07e3f0dcd582a9a6c1?d=https://a248.e.akamai.net/assets.github.com%2Fimages%2Fgravatars%2Fgravatar-140.png","login":"Jakobo"},"repo":{"created_at":"2011-09-08T00:38:32Z","pushed_at":"2011-11-29T01:50:23Z","updated_at":"2011-11-29T01:50:24Z","private":false,"clone_url":"https://github.com/Jakobo/inject.git","name":"inject","master_branch":null,"watchers":10,"svn_url":"https://svn.github.com/Jakobo/inject","language":"JavaScript","forks":6,"owner":{"gravatar_id":"e7df5aa388c83e07e3f0dcd582a9a6c1","id":1795,"url":"https://api.github.com/users/Jakobo","avatar_url":"https://secure.gravatar.com/avatar/e7df5aa388c83e07e3f0dcd582a9a6c1?d=https://a248.e.akamai.net/assets.github.com%2Fimages%2Fgravatars%2Fgravatar-140.png","login":"Jakobo"},"id":2345470,"homepage":"","git_url":"git://github.com/Jakobo/inject.git","description":"Dependency Management got Awesome","html_url":"https://github.com/Jakobo/inject","size":284,"open_issues":10,"fork":false,"url":"https://api.github.com/repos/Jakobo/inject","ssh_url":"git@github.com:Jakobo/inject.git"},"sha":"336caf2babcdb9a71157a2880231d5fac849276d","ref":"addrule","label":"Jakobo:addrule"},"title":"addRule prototype","closed_at":null,"html_url":"https://github.com/Jakobo/inject/pull/37","url":"https://api.github.com/repos/Jakobo/inject/pulls/37","diff_url":"https://github.com/Jakobo/inject/pull/37.diff","merged_at":null}}