{"comment":{"url":"https://api.github.com/repos/acuminous/yadda/comments/12168024","html_url":"https://github.com/acuminous/yadda/commit/63e0f51e1fd5bfe5d1a3e42e2ce478cfef2c1fc2#commitcomment-12168024","id":12168024,"user":{"login":"cressie176","id":229672,"avatar_url":"https://avatars.githubusercontent.com/u/229672?v=3","gravatar_id":"","url":"https://api.github.com/users/cressie176","html_url":"https://github.com/cressie176","followers_url":"https://api.github.com/users/cressie176/followers","following_url":"https://api.github.com/users/cressie176/following{/other_user}","gists_url":"https://api.github.com/users/cressie176/gists{/gist_id}","starred_url":"https://api.github.com/users/cressie176/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/cressie176/subscriptions","organizations_url":"https://api.github.com/users/cressie176/orgs","repos_url":"https://api.github.com/users/cressie176/repos","events_url":"https://api.github.com/users/cressie176/events{/privacy}","received_events_url":"https://api.github.com/users/cressie176/received_events","type":"User","site_admin":false},"position":null,"line":null,"path":"","commit_id":"63e0f51e1fd5bfe5d1a3e42e2ce478cfef2c1fc2","created_at":"2015-07-14T23:26:13Z","updated_at":"2015-07-14T23:26:13Z","body":"Fair question. Three reasons\r\n\r\n1. The PR contains multiple changes (e.g. annotations on examples, trimming whitespace). It's possible some of these changes may not be appropriate, or (as in the case of blank line separators vs dash example separators) there might be a preferable alternative.\r\n\r\n2. I sometimes use Yadda to demonstrate that it's possible to write software without else statements, switch statements or mutators functions, with minimal accessor functions and an average function size of < 4 loc. (See https://github.com/acuminous/yadda/wiki/Notes-for-Contributors for more details). The PR included some medium length functions and relies on switch statements so I want to refactor. \r\n\r\n3. Quite some time ago @ami44 created an [issue](https://github.com/acuminous/yadda/issues/53) for docstrings. I think I can implement this using the same approach for multiline examples,\r\n\r\nNone of this is to say that I don't really appreciate your contribution. I think you've done an excellent job implementing a tricky feature and have considered several edge cases.\r\n\r\n"}}