{"action":"opened","issue":{"url":"https://api.github.com/repos/FIRST-Team-1699/ini-reader/issues/6","repository_url":"https://api.github.com/repos/FIRST-Team-1699/ini-reader","labels_url":"https://api.github.com/repos/FIRST-Team-1699/ini-reader/issues/6/labels{/name}","comments_url":"https://api.github.com/repos/FIRST-Team-1699/ini-reader/issues/6/comments","events_url":"https://api.github.com/repos/FIRST-Team-1699/ini-reader/issues/6/events","html_url":"https://github.com/FIRST-Team-1699/ini-reader/issues/6","id":201481297,"number":6,"title":"Refactor readFile","user":{"login":"seth-crampton","id":5529486,"avatar_url":"https://avatars.githubusercontent.com/u/5529486?v=3","gravatar_id":"","url":"https://api.github.com/users/seth-crampton","html_url":"https://github.com/seth-crampton","followers_url":"https://api.github.com/users/seth-crampton/followers","following_url":"https://api.github.com/users/seth-crampton/following{/other_user}","gists_url":"https://api.github.com/users/seth-crampton/gists{/gist_id}","starred_url":"https://api.github.com/users/seth-crampton/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/seth-crampton/subscriptions","organizations_url":"https://api.github.com/users/seth-crampton/orgs","repos_url":"https://api.github.com/users/seth-crampton/repos","events_url":"https://api.github.com/users/seth-crampton/events{/privacy}","received_events_url":"https://api.github.com/users/seth-crampton/received_events","type":"User","site_admin":false},"labels":[{"id":319121473,"url":"https://api.github.com/repos/FIRST-Team-1699/ini-reader/labels/enhancement","name":"enhancement","color":"84b6eb","default":true},{"id":319121474,"url":"https://api.github.com/repos/FIRST-Team-1699/ini-reader/labels/help%20wanted","name":"help wanted","color":"159818","default":true}],"state":"open","locked":false,"assignee":{"login":"seth-crampton","id":5529486,"avatar_url":"https://avatars.githubusercontent.com/u/5529486?v=3","gravatar_id":"","url":"https://api.github.com/users/seth-crampton","html_url":"https://github.com/seth-crampton","followers_url":"https://api.github.com/users/seth-crampton/followers","following_url":"https://api.github.com/users/seth-crampton/following{/other_user}","gists_url":"https://api.github.com/users/seth-crampton/gists{/gist_id}","starred_url":"https://api.github.com/users/seth-crampton/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/seth-crampton/subscriptions","organizations_url":"https://api.github.com/users/seth-crampton/orgs","repos_url":"https://api.github.com/users/seth-crampton/repos","events_url":"https://api.github.com/users/seth-crampton/events{/privacy}","received_events_url":"https://api.github.com/users/seth-crampton/received_events","type":"User","site_admin":false},"assignees":[{"login":"seth-crampton","id":5529486,"avatar_url":"https://avatars.githubusercontent.com/u/5529486?v=3","gravatar_id":"","url":"https://api.github.com/users/seth-crampton","html_url":"https://github.com/seth-crampton","followers_url":"https://api.github.com/users/seth-crampton/followers","following_url":"https://api.github.com/users/seth-crampton/following{/other_user}","gists_url":"https://api.github.com/users/seth-crampton/gists{/gist_id}","starred_url":"https://api.github.com/users/seth-crampton/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/seth-crampton/subscriptions","organizations_url":"https://api.github.com/users/seth-crampton/orgs","repos_url":"https://api.github.com/users/seth-crampton/repos","events_url":"https://api.github.com/users/seth-crampton/events{/privacy}","received_events_url":"https://api.github.com/users/seth-crampton/received_events","type":"User","site_admin":false},{"login":"squirlemaster42","id":5611164,"avatar_url":"https://avatars.githubusercontent.com/u/5611164?v=3","gravatar_id":"","url":"https://api.github.com/users/squirlemaster42","html_url":"https://github.com/squirlemaster42","followers_url":"https://api.github.com/users/squirlemaster42/followers","following_url":"https://api.github.com/users/squirlemaster42/following{/other_user}","gists_url":"https://api.github.com/users/squirlemaster42/gists{/gist_id}","starred_url":"https://api.github.com/users/squirlemaster42/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/squirlemaster42/subscriptions","organizations_url":"https://api.github.com/users/squirlemaster42/orgs","repos_url":"https://api.github.com/users/squirlemaster42/repos","events_url":"https://api.github.com/users/squirlemaster42/events{/privacy}","received_events_url":"https://api.github.com/users/squirlemaster42/received_events","type":"User","site_admin":false},{"login":"thatging3rkid","id":14779972,"avatar_url":"https://avatars.githubusercontent.com/u/14779972?v=3","gravatar_id":"","url":"https://api.github.com/users/thatging3rkid","html_url":"https://github.com/thatging3rkid","followers_url":"https://api.github.com/users/thatging3rkid/followers","following_url":"https://api.github.com/users/thatging3rkid/following{/other_user}","gists_url":"https://api.github.com/users/thatging3rkid/gists{/gist_id}","starred_url":"https://api.github.com/users/thatging3rkid/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/thatging3rkid/subscriptions","organizations_url":"https://api.github.com/users/thatging3rkid/orgs","repos_url":"https://api.github.com/users/thatging3rkid/repos","events_url":"https://api.github.com/users/thatging3rkid/events{/privacy}","received_events_url":"https://api.github.com/users/thatging3rkid/received_events","type":"User","site_admin":false}],"milestone":null,"comments":0,"created_at":"2017-01-18T05:14:58Z","updated_at":"2017-01-18T05:14:58Z","closed_at":null,"body":"So 200 lines of code is a bit excessive for a single method.  We should at least create helper methods that make the code more readable. I was thinking we would pull the parsing out of the lines out of the method. That might require us to make rawConfigSection and/or rawConfigLine objects that hold unparsed versions or ConfigSection and/or ConfigLine objects. \r\n\r\nWe could also create new classes to specifically handle building up the ConfigSecion and ConfigLine Objects. Following the [Builder Design Pattern](https://en.wikipedia.org/wiki/Builder_pattern) should help implement it this way . \r\n\r\nThis is a low priority but I might make a new branch to try to clean it up and make it more manageable without killing it. "}}
{"actor":{"display_login":"seth-crampton"}}