{"action":"created","issue":{"url":"https://api.github.com/repos/ethereum/webthree-umbrella/issues/486","repository_url":"https://api.github.com/repos/ethereum/webthree-umbrella","labels_url":"https://api.github.com/repos/ethereum/webthree-umbrella/issues/486/labels{/name}","comments_url":"https://api.github.com/repos/ethereum/webthree-umbrella/issues/486/comments","events_url":"https://api.github.com/repos/ethereum/webthree-umbrella/issues/486/events","html_url":"https://github.com/ethereum/webthree-umbrella/issues/486","id":151254652,"number":486,"title":"For our coding standards to be meaningful, they need to be automatable","user":{"login":"bobsummerwill","id":3788156,"avatar_url":"https://avatars.githubusercontent.com/u/3788156?v=3","gravatar_id":"","url":"https://api.github.com/users/bobsummerwill","html_url":"https://github.com/bobsummerwill","followers_url":"https://api.github.com/users/bobsummerwill/followers","following_url":"https://api.github.com/users/bobsummerwill/following{/other_user}","gists_url":"https://api.github.com/users/bobsummerwill/gists{/gist_id}","starred_url":"https://api.github.com/users/bobsummerwill/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/bobsummerwill/subscriptions","organizations_url":"https://api.github.com/users/bobsummerwill/orgs","repos_url":"https://api.github.com/users/bobsummerwill/repos","events_url":"https://api.github.com/users/bobsummerwill/events{/privacy}","received_events_url":"https://api.github.com/users/bobsummerwill/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/ethereum/webthree-umbrella/labels/enhancement","name":"enhancement","color":"84b6eb"}],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":4,"created_at":"2016-04-26T22:29:57Z","updated_at":"2016-04-26T23:37:15Z","closed_at":null,"body":"That probably means either `astyle` or `clang-format`, and we need the ability to run it locally and to have it run as part of our automation.\r\n\r\nSo probably `make style` to report on violations, lint-style, and `make fix-style` to automatically fix the violations in-place if that is possible.\r\n\r\nI **do not** think it would be a good idea for the automation to do this automatically, because that runs the risk of the automation introducing issues into the code which then have not ever been tested.    Instead, this would be in the form of an additional gate in the PR process.\r\n"},"comment":{"url":"https://api.github.com/repos/ethereum/webthree-umbrella/issues/comments/214920095","html_url":"https://github.com/ethereum/webthree-umbrella/issues/486#issuecomment-214920095","issue_url":"https://api.github.com/repos/ethereum/webthree-umbrella/issues/486","id":214920095,"user":{"login":"gcolvin","id":16827129,"avatar_url":"https://avatars.githubusercontent.com/u/16827129?v=3","gravatar_id":"","url":"https://api.github.com/users/gcolvin","html_url":"https://github.com/gcolvin","followers_url":"https://api.github.com/users/gcolvin/followers","following_url":"https://api.github.com/users/gcolvin/following{/other_user}","gists_url":"https://api.github.com/users/gcolvin/gists{/gist_id}","starred_url":"https://api.github.com/users/gcolvin/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/gcolvin/subscriptions","organizations_url":"https://api.github.com/users/gcolvin/orgs","repos_url":"https://api.github.com/users/gcolvin/repos","events_url":"https://api.github.com/users/gcolvin/events{/privacy}","received_events_url":"https://api.github.com/users/gcolvin/received_events","type":"User","site_admin":false},"created_at":"2016-04-26T23:37:15Z","updated_at":"2016-04-26T23:37:15Z","body":"From what I can tell neither tool has a diff-only mode, but as we discussed one can be created by diffing its output with its input.  I don't know which tool is better, but astyle is better documented, and there is a start at https://github.com/ethereum/webthree-umbrella/blob/develop/astylerc that I notice Paweł has contributed to.  However, Paweł is right that .clang-format can simply be dropped into the top-level directory of a repo, whereas .astylerc must be referenced by an environment variable or on the command-line."}}