{"action":"created","issue":{"url":"https://api.github.com/repos/terraform-aws-modules/terraform-aws-eks/issues/211","repository_url":"https://api.github.com/repos/terraform-aws-modules/terraform-aws-eks","labels_url":"https://api.github.com/repos/terraform-aws-modules/terraform-aws-eks/issues/211/labels{/name}","comments_url":"https://api.github.com/repos/terraform-aws-modules/terraform-aws-eks/issues/211/comments","events_url":"https://api.github.com/repos/terraform-aws-modules/terraform-aws-eks/issues/211/events","html_url":"https://github.com/terraform-aws-modules/terraform-aws-eks/pull/211","id":390334840,"node_id":"MDExOlB1bGxSZXF1ZXN0MjM4MTIzNzEy","number":211,"title":"Use sed instead of tail for terraform-docs command","user":{"login":"RothAndrew","id":16000938,"node_id":"MDQ6VXNlcjE2MDAwOTM4","avatar_url":"https://avatars0.githubusercontent.com/u/16000938?v=4","gravatar_id":"","url":"https://api.github.com/users/RothAndrew","html_url":"https://github.com/RothAndrew","followers_url":"https://api.github.com/users/RothAndrew/followers","following_url":"https://api.github.com/users/RothAndrew/following{/other_user}","gists_url":"https://api.github.com/users/RothAndrew/gists{/gist_id}","starred_url":"https://api.github.com/users/RothAndrew/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/RothAndrew/subscriptions","organizations_url":"https://api.github.com/users/RothAndrew/orgs","repos_url":"https://api.github.com/users/RothAndrew/repos","events_url":"https://api.github.com/users/RothAndrew/events{/privacy}","received_events_url":"https://api.github.com/users/RothAndrew/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":5,"created_at":"2018-12-12T17:39:06Z","updated_at":"2018-12-13T17:20:46Z","closed_at":null,"author_association":"CONTRIBUTOR","pull_request":{"url":"https://api.github.com/repos/terraform-aws-modules/terraform-aws-eks/pulls/211","html_url":"https://github.com/terraform-aws-modules/terraform-aws-eks/pull/211","diff_url":"https://github.com/terraform-aws-modules/terraform-aws-eks/pull/211.diff","patch_url":"https://github.com/terraform-aws-modules/terraform-aws-eks/pull/211.patch"},"body":"# PR o'clock\r\n\r\n## Description\r\n\r\nUse `sed` instead of `tail` for the `terraform-docs` command since it is more universally compatible. Thanks to @dpiddockcmp for your suggestion at https://github.com/terraform-aws-modules/terraform-aws-eks/pull/209#issuecomment-446671578\r\n\r\n### Checklist\r\n\r\n- [x] `terraform fmt` and `terraform validate` both work from the root and `examples/eks_test_fixture` directories (look in CI for an example)\r\n- [ ] Tests for the changes have been added and passing (for bug fixes/features)\r\n- [ ] Test results are pasted in this PR (in lieu of CI)\r\n- [x] Docs have been updated using `terraform-docs` per `README.md` instructions\r\n- [ ] I've added my change to CHANGELOG.md\r\n- [x] Any breaking changes are highlighted above\r\n"},"comment":{"url":"https://api.github.com/repos/terraform-aws-modules/terraform-aws-eks/issues/comments/447050398","html_url":"https://github.com/terraform-aws-modules/terraform-aws-eks/pull/211#issuecomment-447050398","issue_url":"https://api.github.com/repos/terraform-aws-modules/terraform-aws-eks/issues/211","id":447050398,"node_id":"MDEyOklzc3VlQ29tbWVudDQ0NzA1MDM5OA==","user":{"login":"RothAndrew","id":16000938,"node_id":"MDQ6VXNlcjE2MDAwOTM4","avatar_url":"https://avatars0.githubusercontent.com/u/16000938?v=4","gravatar_id":"","url":"https://api.github.com/users/RothAndrew","html_url":"https://github.com/RothAndrew","followers_url":"https://api.github.com/users/RothAndrew/followers","following_url":"https://api.github.com/users/RothAndrew/following{/other_user}","gists_url":"https://api.github.com/users/RothAndrew/gists{/gist_id}","starred_url":"https://api.github.com/users/RothAndrew/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/RothAndrew/subscriptions","organizations_url":"https://api.github.com/users/RothAndrew/orgs","repos_url":"https://api.github.com/users/RothAndrew/repos","events_url":"https://api.github.com/users/RothAndrew/events{/privacy}","received_events_url":"https://api.github.com/users/RothAndrew/received_events","type":"User","site_admin":false},"created_at":"2018-12-13T17:20:46Z","updated_at":"2018-12-13T17:20:46Z","author_association":"CONTRIBUTOR","body":"> @antonbabenko\r\n> \r\n> > Guys, I may sound strange, but why don't you want to use the same pre-commit-terraform which I am using in other terraform-aws-modules.\r\n> \r\n> Sure thing, let's do it \r\n> \r\n> Will it work on multiple platforms like what this PR is solving?\r\n> \r\n> > Let me know if you want me to help with it and I can open a PR.\r\n> \r\n> Please do!\r\n\r\nBoth options should be available and equivalent. The pre-commit needs to be installed locally for it to work."}}
{"actor":{"display_login":"RothAndrew"}}