{"action":"created","issue":{"url":"https://api.github.com/repos/nim-lang/Nim/issues/2967","labels_url":"https://api.github.com/repos/nim-lang/Nim/issues/2967/labels{/name}","comments_url":"https://api.github.com/repos/nim-lang/Nim/issues/2967/comments","events_url":"https://api.github.com/repos/nim-lang/Nim/issues/2967/events","html_url":"https://github.com/nim-lang/Nim/issues/2967","id":89927753,"number":2967,"title":"Trailing whitespace cleanup in a targeted manner","user":{"login":"Perelandric","id":1114646,"avatar_url":"https://avatars.githubusercontent.com/u/1114646?v=3","gravatar_id":"","url":"https://api.github.com/users/Perelandric","html_url":"https://github.com/Perelandric","followers_url":"https://api.github.com/users/Perelandric/followers","following_url":"https://api.github.com/users/Perelandric/following{/other_user}","gists_url":"https://api.github.com/users/Perelandric/gists{/gist_id}","starred_url":"https://api.github.com/users/Perelandric/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Perelandric/subscriptions","organizations_url":"https://api.github.com/users/Perelandric/orgs","repos_url":"https://api.github.com/users/Perelandric/repos","events_url":"https://api.github.com/users/Perelandric/events{/privacy}","received_events_url":"https://api.github.com/users/Perelandric/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/nim-lang/Nim/labels/RFC","name":"RFC","color":"1d1d1d"}],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":6,"created_at":"2015-06-21T16:05:33Z","updated_at":"2015-06-21T18:28:25Z","closed_at":null,"body":"Stemming from a discussion with @Araq, @dom96, @nanoant, @kirbyfan64 and @reactormonk, it seems that while there's a need/desire to clean up trailing white-spaces in `.nim` files, it can cause headache if not done in a sufficiently accommodating manner.\r\n\r\nMy suggestion is to take somewhat of a divide-and-conquer approach to the problem...\r\n\r\n - Contributors will list under this issue any files that ought to be \"hands-off\" due to potential conflicts.\r\n - This issue will be resolved via a series of PRs, each of which will target an orderly yet somewhat arbitrarily defined group of files.\r\n - Each PR will have a separate commit for every individual file in the group with its trailing white-space removed. The commits will include the filepath for clarity.\r\n - There will be a short period of time during which contributors may object to any of the individual commits because of conflicts not previously noted here. The objections will be excluded from the eventual merge.\r\n\r\nIt would seem this would ameliorate hopefully most of the problems arising from this issue while getting us much of the way toward the cleanup being complete. \r\n\r\nI'd volunteer to do the work and put together the PRs if this seems like a good idea. I'll let others decide some of the details of how long the \"objection period\" should last and how it should be announced.\r\n\r\n----\r\n*EDIT:* As noted below, this would result in far too many commits. More likely we would just give contributors the opportunity to declare \"hands-off\" files and then do it all at once with per project folder commits."},"comment":{"url":"https://api.github.com/repos/nim-lang/Nim/issues/comments/113938250","html_url":"https://github.com/nim-lang/Nim/issues/2967#issuecomment-113938250","issue_url":"https://api.github.com/repos/nim-lang/Nim/issues/2967","id":113938250,"user":{"login":"reactormonk","id":49185,"avatar_url":"https://avatars.githubusercontent.com/u/49185?v=3","gravatar_id":"","url":"https://api.github.com/users/reactormonk","html_url":"https://github.com/reactormonk","followers_url":"https://api.github.com/users/reactormonk/followers","following_url":"https://api.github.com/users/reactormonk/following{/other_user}","gists_url":"https://api.github.com/users/reactormonk/gists{/gist_id}","starred_url":"https://api.github.com/users/reactormonk/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/reactormonk/subscriptions","organizations_url":"https://api.github.com/users/reactormonk/orgs","repos_url":"https://api.github.com/users/reactormonk/repos","events_url":"https://api.github.com/users/reactormonk/events{/privacy}","received_events_url":"https://api.github.com/users/reactormonk/received_events","type":"User","site_admin":false},"created_at":"2015-06-21T18:28:25Z","updated_at":"2015-06-21T18:28:25Z","body":"I'm all for purges."}}