{"action":"created","issue":{"url":"https://api.github.com/repos/emacs-ess/ESS/issues/195","labels_url":"https://api.github.com/repos/emacs-ess/ESS/issues/195/labels{/name}","comments_url":"https://api.github.com/repos/emacs-ess/ESS/issues/195/comments","events_url":"https://api.github.com/repos/emacs-ess/ESS/issues/195/events","html_url":"https://github.com/emacs-ess/ESS/pull/195","id":92650223,"number":195,"title":"Fix #190 #192","user":{"login":"lionel-","id":4465050,"avatar_url":"https://avatars.githubusercontent.com/u/4465050?v=3","gravatar_id":"","url":"https://api.github.com/users/lionel-","html_url":"https://github.com/lionel-","followers_url":"https://api.github.com/users/lionel-/followers","following_url":"https://api.github.com/users/lionel-/following{/other_user}","gists_url":"https://api.github.com/users/lionel-/gists{/gist_id}","starred_url":"https://api.github.com/users/lionel-/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/lionel-/subscriptions","organizations_url":"https://api.github.com/users/lionel-/orgs","repos_url":"https://api.github.com/users/lionel-/repos","events_url":"https://api.github.com/users/lionel-/events{/privacy}","received_events_url":"https://api.github.com/users/lionel-/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":7,"created_at":"2015-07-02T14:21:20Z","updated_at":"2015-07-04T12:59:29Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/emacs-ess/ESS/pulls/195","html_url":"https://github.com/emacs-ess/ESS/pull/195","diff_url":"https://github.com/emacs-ess/ESS/pull/195.diff","patch_url":"https://github.com/emacs-ess/ESS/pull/195.patch"},"body":"Here are the updates for the last discussions including the one in #189.\r\n\r\nAbout the overrides for blocks, I implemented it as a list because it was easy enough, and on second thought I think it can be useful. This way we allow both if-else blocks and function declaration blocks to be aligned vertically. It makes sense that some users may want indentation at prev-line of blocks in general (for the `test_that` DSL for example) but still want their lambda functions to be indented in a compact way. In particular, some users like to have code in function arguments (cf Martin's example with an if-else block as default function argument), and it looks better to have vertical alignment in this case.\r\n\r\nBut if you object I can change it to a boolean."},"comment":{"url":"https://api.github.com/repos/emacs-ess/ESS/issues/comments/118511151","html_url":"https://github.com/emacs-ess/ESS/pull/195#issuecomment-118511151","issue_url":"https://api.github.com/repos/emacs-ess/ESS/issues/195","id":118511151,"user":{"login":"vspinu","id":1363467,"avatar_url":"https://avatars.githubusercontent.com/u/1363467?v=3","gravatar_id":"","url":"https://api.github.com/users/vspinu","html_url":"https://github.com/vspinu","followers_url":"https://api.github.com/users/vspinu/followers","following_url":"https://api.github.com/users/vspinu/following{/other_user}","gists_url":"https://api.github.com/users/vspinu/gists{/gist_id}","starred_url":"https://api.github.com/users/vspinu/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/vspinu/subscriptions","organizations_url":"https://api.github.com/users/vspinu/orgs","repos_url":"https://api.github.com/users/vspinu/repos","events_url":"https://api.github.com/users/vspinu/events{/privacy}","received_events_url":"https://api.github.com/users/vspinu/received_events","type":"User","site_admin":false},"created_at":"2015-07-04T12:59:29Z","updated_at":"2015-07-04T12:59:29Z","body":"Ok. I am clarified now. We basically converged on having `ess-indent-align-xxx` to fine tune other offsets in a specific vertical pattern.  That's not bad, I think."}}