{"push_id":1197131826,"size":2,"distinct_size":1,"ref":"refs/heads/master","head":"a69587f64d923603b27fc711efad56f7842a95a8","before":"875ddc9929d79279c888761e96b5dd474b89dd02","commits":[{"sha":"db775088d0efafb23c725ca616a589dca58304f4","author":{"name":"Giorgio Sironi","email":"c19a24341773c20dcd23002332881d67e4692a60@elifesciences.org"},"message":"Recovers from a command being passed a non-existent stackname on the cli\n\nAfter the `stackname` is not found in the list and asked to the user, the wrong value has to be removed from `args` to avoid passing it twice with two different values:\n```\n(venv) [09:51:39][giorgio@Newton:~/code/builder]$ ./bldr aws_update_stack:does_not_exist\n1467967958.687872 - WARNING - MainProcess - buildercore.decorators - TODO: embarassing code. some of these constants should be pulled form project config or given better names.\n1467967958.710811 - INFO - MainProcess - buildercore.config - using settings path u'/home/giorgio/code/builder/settings.yml'\n1467967959.244435 - WARNING - MainProcess - buildercore.decorators - TODO: `raw_aws_stacks` duplicate code/unclear differences. .list_stacks with filter vs .describe_stacks with stackname\n1467967964.498469 - WARNING - MainProcess - buildercore.decorators - TODO: `_pick` renamed from `_pick` to something. `choose` ?\n\nplease pick a known stack:\n1 - api-gateway--2016-07-01\n2 - api-gateway--end2end\n3 - elife-bot--2016-07-05\n4 - elife-metrics--2016-07-04\n5 - journal--master\n6 - journal-cms--master\n7 - lax--2016-06-27\n8 - master-server--temp\n> ('api-gateway--end2end') 2\nTraceback (most recent call last):\n  File \"/home/giorgio/code/builder/venv/local/lib/python2.7/site-packages/fabric/main.py\", line 745, in main\n    *args, **kwargs\n  File \"/home/giorgio/code/builder/venv/local/lib/python2.7/site-packages/fabric/tasks.py\", line 427, in execute\n    results['<local-only>'] = task.run(*args, **new_kwargs)\n  File \"/home/giorgio/code/builder/venv/local/lib/python2.7/site-packages/fabric/tasks.py\", line 174, in run\n    return self.wrapped(*args, **kwargs)\n  File \"/home/giorgio/code/builder/src/decorators.py\", line 103, in call\n    return func(stackname, *args, **kwargs)\nTypeError: update() takes exactly 1 argument (2 given)\n```","distinct":false,"url":"https://api.github.com/repos/elifesciences/builder/commits/db775088d0efafb23c725ca616a589dca58304f4"},{"sha":"a69587f64d923603b27fc711efad56f7842a95a8","author":{"name":"Luke Skibinski","email":"4da9e90a91a064b78db0cacdaa3e3f7a01fbd7e4@users.noreply.github.com"},"message":"Merge pull request #11 from elifesciences/recovering_from_wrong_stack_name\n\nRecovers from a command being passed a non-existent stackname on the cli","distinct":true,"url":"https://api.github.com/repos/elifesciences/builder/commits/a69587f64d923603b27fc711efad56f7842a95a8"}]}
{"actor":{"display_login":"lsh-0"}}