{"action":"created","issue":{"url":"https://api.github.com/repos/larshp/abapGit/issues/419","repository_url":"https://api.github.com/repos/larshp/abapGit","labels_url":"https://api.github.com/repos/larshp/abapGit/issues/419/labels{/name}","comments_url":"https://api.github.com/repos/larshp/abapGit/issues/419/comments","events_url":"https://api.github.com/repos/larshp/abapGit/issues/419/events","html_url":"https://github.com/larshp/abapGit/pull/419","id":188908832,"number":419,"title":"Independent diffs, local state per file, refactorings","user":{"login":"sbcgua","id":15635498,"avatar_url":"https://avatars.githubusercontent.com/u/15635498?v=3","gravatar_id":"","url":"https://api.github.com/users/sbcgua","html_url":"https://github.com/sbcgua","followers_url":"https://api.github.com/users/sbcgua/followers","following_url":"https://api.github.com/users/sbcgua/following{/other_user}","gists_url":"https://api.github.com/users/sbcgua/gists{/gist_id}","starred_url":"https://api.github.com/users/sbcgua/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/sbcgua/subscriptions","organizations_url":"https://api.github.com/users/sbcgua/orgs","repos_url":"https://api.github.com/users/sbcgua/repos","events_url":"https://api.github.com/users/sbcgua/events{/privacy}","received_events_url":"https://api.github.com/users/sbcgua/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":10,"created_at":"2016-11-12T12:18:49Z","updated_at":"2016-11-15T07:05:46Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/larshp/abapGit/pulls/419","html_url":"https://github.com/larshp/abapGit/pull/419","diff_url":"https://github.com/larshp/abapGit/pull/419.diff","patch_url":"https://github.com/larshp/abapGit/pull/419.patch"},"body":"To #407 \r\n\r\nSorry for huge PR. The issue was not superficial at all. Had to refactor much internals. Here is come comment for history and as a help for review.\r\n\r\n- changelog.txt added\r\n- INCLUDE zabapgit_migrations added - called from run form, contains migration methods and call, probably a good place for future migrations\r\n- Definitions: ty_file_signature added as a shorter vehicle for hash data, gc_state - list of file states\r\n- zabapgit_file_status - completely rewritten to support individual state calculation and performance. Hopefully has better structure now. All files in the result has now remote and local states, path and filename (needed for whole diff in particular).\r\n- zabapgit_git.prog - push returns the list of actually pushed files\r\n- zabapgit_objects.prog - lcl_objects_files collects accessed files which after is used to update only those checksums which were actually pulled\r\n- page_main and other UI classes - styles and code for independent diff display\r\n- zabapgit_repo.prog - build_local_checksums removed, instead rebuild_local_checksums for complete guess-rebuild, and update_local_checksums to update specific filelist (actually pulled/pushed). Update does not recalc sha1 - it relyes on proper param input. \r\n- service_repo - can refresh_local_checksums\r\n- zabapgit_tadir.prog - got helper method to find object package (without all tadir struc)\r\n- unit test for file_state\r\n\r\n"},"comment":{"url":"https://api.github.com/repos/larshp/abapGit/issues/comments/260563911","html_url":"https://github.com/larshp/abapGit/pull/419#issuecomment-260563911","issue_url":"https://api.github.com/repos/larshp/abapGit/issues/419","id":260563911,"user":{"login":"larshp","id":5888506,"avatar_url":"https://avatars.githubusercontent.com/u/5888506?v=3","gravatar_id":"","url":"https://api.github.com/users/larshp","html_url":"https://github.com/larshp","followers_url":"https://api.github.com/users/larshp/followers","following_url":"https://api.github.com/users/larshp/following{/other_user}","gists_url":"https://api.github.com/users/larshp/gists{/gist_id}","starred_url":"https://api.github.com/users/larshp/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/larshp/subscriptions","organizations_url":"https://api.github.com/users/larshp/orgs","repos_url":"https://api.github.com/users/larshp/repos","events_url":"https://api.github.com/users/larshp/events{/privacy}","received_events_url":"https://api.github.com/users/larshp/received_events","type":"User","site_admin":false},"created_at":"2016-11-15T07:05:46Z","updated_at":"2016-11-15T07:05:46Z","body":"okay, we'll fix the coloring of the diff in next PR, I'll open a separate issue for it\r\n\r\nsquashing and merging\r\n\r\nthanks :smile: "}}
{"actor":{"display_login":"larshp"}}