{"action":"closed","issue":{"url":"https://api.github.com/repos/UAVCAN/uavcan/issues/32","labels_url":"https://api.github.com/repos/UAVCAN/uavcan/issues/32/labels{/name}","comments_url":"https://api.github.com/repos/UAVCAN/uavcan/issues/32/comments","events_url":"https://api.github.com/repos/UAVCAN/uavcan/issues/32/events","html_url":"https://github.com/UAVCAN/uavcan/issues/32","id":81082096,"number":32,"title":"POSIX tools refactoring","user":{"login":"pavel-kirienko","id":3298404,"avatar_url":"https://avatars.githubusercontent.com/u/3298404?v=3","gravatar_id":"","url":"https://api.github.com/users/pavel-kirienko","html_url":"https://github.com/pavel-kirienko","followers_url":"https://api.github.com/users/pavel-kirienko/followers","following_url":"https://api.github.com/users/pavel-kirienko/following{/other_user}","gists_url":"https://api.github.com/users/pavel-kirienko/gists{/gist_id}","starred_url":"https://api.github.com/users/pavel-kirienko/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/pavel-kirienko/subscriptions","organizations_url":"https://api.github.com/users/pavel-kirienko/orgs","repos_url":"https://api.github.com/users/pavel-kirienko/repos","events_url":"https://api.github.com/users/pavel-kirienko/events{/privacy}","received_events_url":"https://api.github.com/users/pavel-kirienko/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":{"login":"davids5","id":1945821,"avatar_url":"https://avatars.githubusercontent.com/u/1945821?v=3","gravatar_id":"","url":"https://api.github.com/users/davids5","html_url":"https://github.com/davids5","followers_url":"https://api.github.com/users/davids5/followers","following_url":"https://api.github.com/users/davids5/following{/other_user}","gists_url":"https://api.github.com/users/davids5/gists{/gist_id}","starred_url":"https://api.github.com/users/davids5/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/davids5/subscriptions","organizations_url":"https://api.github.com/users/davids5/orgs","repos_url":"https://api.github.com/users/davids5/repos","events_url":"https://api.github.com/users/davids5/events{/privacy}","received_events_url":"https://api.github.com/users/davids5/received_events","type":"User","site_admin":false},"milestone":null,"comments":3,"created_at":"2015-05-26T18:34:28Z","updated_at":"2015-05-27T16:42:05Z","closed_at":"2015-05-27T16:42:05Z","body":"@davids5 I reviewed the firmware file management logic in the POSIX tools module, and introduced some style fixes and comments. I believe the classes `FirmwareCommon` and `FirmwareTools` should be removed, and their logic should be moved inside `FirmwareVersionChecker`. Please see the TODO comments I placed in the code.\r\n\r\nThanks!"}}