{"action":"created","issue":{"url":"https://api.github.com/repos/adammck/terraform-inventory/issues/5","labels_url":"https://api.github.com/repos/adammck/terraform-inventory/issues/5/labels{/name}","comments_url":"https://api.github.com/repos/adammck/terraform-inventory/issues/5/comments","events_url":"https://api.github.com/repos/adammck/terraform-inventory/issues/5/events","html_url":"https://github.com/adammck/terraform-inventory/pull/5","id":81580056,"number":5,"title":"Patch to add support for environment variables","user":{"login":"mbainter","id":123376,"avatar_url":"https://avatars.githubusercontent.com/u/123376?v=3","gravatar_id":"","url":"https://api.github.com/users/mbainter","html_url":"https://github.com/mbainter","followers_url":"https://api.github.com/users/mbainter/followers","following_url":"https://api.github.com/users/mbainter/following{/other_user}","gists_url":"https://api.github.com/users/mbainter/gists{/gist_id}","starred_url":"https://api.github.com/users/mbainter/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mbainter/subscriptions","organizations_url":"https://api.github.com/users/mbainter/orgs","repos_url":"https://api.github.com/users/mbainter/repos","events_url":"https://api.github.com/users/mbainter/events{/privacy}","received_events_url":"https://api.github.com/users/mbainter/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2015-05-27T19:02:37Z","updated_at":"2015-05-27T19:13:44Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/adammck/terraform-inventory/pulls/5","html_url":"https://github.com/adammck/terraform-inventory/pull/5","diff_url":"https://github.com/adammck/terraform-inventory/pull/5.diff","patch_url":"https://github.com/adammck/terraform-inventory/pull/5.patch"},"body":"This change is admittedly overkill for just defining the state path.\r\nHowever, this opens the door for additional configuration options \r\nto be defined in environment variables such as choosing the keys \r\nto match on for the \"host\" argument."},"comment":{"url":"https://api.github.com/repos/adammck/terraform-inventory/issues/comments/106039211","html_url":"https://github.com/adammck/terraform-inventory/pull/5#issuecomment-106039211","issue_url":"https://api.github.com/repos/adammck/terraform-inventory/issues/5","id":106039211,"user":{"login":"adammck","id":19543,"avatar_url":"https://avatars.githubusercontent.com/u/19543?v=3","gravatar_id":"","url":"https://api.github.com/users/adammck","html_url":"https://github.com/adammck","followers_url":"https://api.github.com/users/adammck/followers","following_url":"https://api.github.com/users/adammck/following{/other_user}","gists_url":"https://api.github.com/users/adammck/gists{/gist_id}","starred_url":"https://api.github.com/users/adammck/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/adammck/subscriptions","organizations_url":"https://api.github.com/users/adammck/orgs","repos_url":"https://api.github.com/users/adammck/repos","events_url":"https://api.github.com/users/adammck/events{/privacy}","received_events_url":"https://api.github.com/users/adammck/received_events","type":"User","site_admin":false},"created_at":"2015-05-27T19:13:44Z","updated_at":"2015-05-27T19:13:44Z","body":"Great, thanks! This is much simpler than using a shell script. Couple of things, though:\r\n\r\n1. Your indentation seems to be screwed up -- you can fix it with `go fmt`\r\n2. Introducing a new dependency for this does seem a bit overkill. Could we just use `os.Getenv`?"}}