{"action":"created","issue":{"url":"https://api.github.com/repos/brianwarner/facade/issues/10","repository_url":"https://api.github.com/repos/brianwarner/facade","labels_url":"https://api.github.com/repos/brianwarner/facade/issues/10/labels{/name}","comments_url":"https://api.github.com/repos/brianwarner/facade/issues/10/comments","events_url":"https://api.github.com/repos/brianwarner/facade/issues/10/events","html_url":"https://github.com/brianwarner/facade/issues/10","id":237238908,"number":10,"title":"Merging Łukasz' fork","user":{"login":"brianwarner","id":1923110,"avatar_url":"https://avatars1.githubusercontent.com/u/1923110?v=4","gravatar_id":"","url":"https://api.github.com/users/brianwarner","html_url":"https://github.com/brianwarner","followers_url":"https://api.github.com/users/brianwarner/followers","following_url":"https://api.github.com/users/brianwarner/following{/other_user}","gists_url":"https://api.github.com/users/brianwarner/gists{/gist_id}","starred_url":"https://api.github.com/users/brianwarner/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/brianwarner/subscriptions","organizations_url":"https://api.github.com/users/brianwarner/orgs","repos_url":"https://api.github.com/users/brianwarner/repos","events_url":"https://api.github.com/users/brianwarner/events{/privacy}","received_events_url":"https://api.github.com/users/brianwarner/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":53,"created_at":"2017-06-20T14:51:50Z","updated_at":"2018-02-27T18:09:45Z","closed_at":"2018-02-27T18:09:45Z","author_association":"OWNER","body":"@lukaszgryglicki,\r\n\r\nI had a look back to where Facade was when you forked it.  Here are a few hints on what's changed since then.\r\n\r\n1) setup.py is much more automated.  It can now create database, users, and tables automatically, and it generates the database connection files automatically.  It can accept user-entered values for the database name, user, and password, or it randomly generates them.\r\n\r\n2) You can now use a shared database for affiliation and alias information.  If you have a few instances of Facade, it is a lot easier when you don't have to keep the affiliation and alias info in sync between them.  This also required some new logic in facade-worker.py to detect changes to affiliation and alias data made by other instances, and allowed me to remove it from the web frontend stuff.  But, it also means that when you import gitdm files or make a change directly in the database, you no longer have to nuke your affiliations.\r\n\r\n3) Some of the tables have changed.  analysis_data now holds the raw author and committer email, as it was in the patch.  These are in author_raw_email and committer_raw_email.  Also, author_email and committer_email now hold the aliased email.  This is so if you delete an alias mapping, we can roll the author/committer email back to what it was in the patch, rather than holding onto it.\r\n\r\n4) Some changes to columns in database tables.  Your best bet is to export your current configs, set up a new instance of facade, and re-import the configs.  This process should work even despite the column changes, as I've tried to keep it from breaking.  However, if it does break, ping me.\r\n\r\n5) By default, git repos clone into a subdirectory of facade now, instead of into /opt.  This seemed to make a lot more sense, reduces the risk of collisions and makes it easy to clean up after a short-lived analysis.\r\n\r\nI think that's it.\r\n\r\n----\r\n\r\nOne request I have when you are merging, please attempt to organize new files into subdirectories as makes sense.  I'm trying to keep the top level as clean as possible for things related to display, and utilities/ for things related to analysis, as these are two different things.  This also makes setting .htaccess rules a little easier.\r\n\r\nAlso, before I release I should update the coding style to PEP 008 (spaces instead of tabs, even though it kills me to do so).  However, I will wait to do this until you've done your merge so we don't end up with more manual merging."},"comment":{"url":"https://api.github.com/repos/brianwarner/facade/issues/comments/368972082","html_url":"https://github.com/brianwarner/facade/issues/10#issuecomment-368972082","issue_url":"https://api.github.com/repos/brianwarner/facade/issues/10","id":368972082,"user":{"login":"brianwarner","id":1923110,"avatar_url":"https://avatars1.githubusercontent.com/u/1923110?v=4","gravatar_id":"","url":"https://api.github.com/users/brianwarner","html_url":"https://github.com/brianwarner","followers_url":"https://api.github.com/users/brianwarner/followers","following_url":"https://api.github.com/users/brianwarner/following{/other_user}","gists_url":"https://api.github.com/users/brianwarner/gists{/gist_id}","starred_url":"https://api.github.com/users/brianwarner/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/brianwarner/subscriptions","organizations_url":"https://api.github.com/users/brianwarner/orgs","repos_url":"https://api.github.com/users/brianwarner/repos","events_url":"https://api.github.com/users/brianwarner/events{/privacy}","received_events_url":"https://api.github.com/users/brianwarner/received_events","type":"User","site_admin":false},"created_at":"2018-02-27T18:09:45Z","updated_at":"2018-02-27T18:09:45Z","author_association":"OWNER","body":"Cleaning up this open but dormant ticket."}}
{"actor":{"display_login":"brianwarner"}}