{"action":"created","issue":{"url":"https://api.github.com/repos/dotnet/roslyn-project-system/issues/847","repository_url":"https://api.github.com/repos/dotnet/roslyn-project-system","labels_url":"https://api.github.com/repos/dotnet/roslyn-project-system/issues/847/labels{/name}","comments_url":"https://api.github.com/repos/dotnet/roslyn-project-system/issues/847/comments","events_url":"https://api.github.com/repos/dotnet/roslyn-project-system/issues/847/events","html_url":"https://github.com/dotnet/roslyn-project-system/pull/847","id":192660705,"number":847,"title":"[WIP] Migration cleanup","user":{"login":"333fred","id":2371880,"avatar_url":"https://avatars.githubusercontent.com/u/2371880?v=3","gravatar_id":"","url":"https://api.github.com/users/333fred","html_url":"https://github.com/333fred","followers_url":"https://api.github.com/users/333fred/followers","following_url":"https://api.github.com/users/333fred/following{/other_user}","gists_url":"https://api.github.com/users/333fred/gists{/gist_id}","starred_url":"https://api.github.com/users/333fred/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/333fred/subscriptions","organizations_url":"https://api.github.com/users/333fred/orgs","repos_url":"https://api.github.com/users/333fred/repos","events_url":"https://api.github.com/users/333fred/events{/privacy}","received_events_url":"https://api.github.com/users/333fred/received_events","type":"User","site_admin":false},"labels":[{"id":370752484,"url":"https://api.github.com/repos/dotnet/roslyn-project-system/labels/cla-already-signed","name":"cla-already-signed","color":"009800","default":false}],"state":"closed","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":5,"created_at":"2016-11-30T19:46:24Z","updated_at":"2016-12-22T02:06:10Z","closed_at":"2016-12-22T02:06:10Z","pull_request":{"url":"https://api.github.com/repos/dotnet/roslyn-project-system/pulls/847","html_url":"https://github.com/dotnet/roslyn-project-system/pull/847","diff_url":"https://github.com/dotnet/roslyn-project-system/pull/847.diff","patch_url":"https://github.com/dotnet/roslyn-project-system/pull/847.patch"},"body":"This fixes #734 by adding all migrated files to a list that the project will clean on load. We can't just delete the xprojs and project.json files after running dotnet migrate because other projects might depend on those files.\r\n\r\nTagging @dotnet/project-system @jinujoseph for review. /cc @piotrpMSFT. \r\n@MattGertz, I'm not sure if this scenario meets the bar for escrow. The scenario is that after migration, we currently leave the old project.json and .xproj files on disk, as well as backing them up. This adds deletion of those files after migration is complete. If you think that meets the bar, I'll retarget to dev15-rc2 and add the full escrow template."},"comment":{"url":"https://api.github.com/repos/dotnet/roslyn-project-system/issues/comments/268699491","html_url":"https://github.com/dotnet/roslyn-project-system/pull/847#issuecomment-268699491","issue_url":"https://api.github.com/repos/dotnet/roslyn-project-system/issues/847","id":268699491,"user":{"login":"333fred","id":2371880,"avatar_url":"https://avatars.githubusercontent.com/u/2371880?v=3","gravatar_id":"","url":"https://api.github.com/users/333fred","html_url":"https://github.com/333fred","followers_url":"https://api.github.com/users/333fred/followers","following_url":"https://api.github.com/users/333fred/following{/other_user}","gists_url":"https://api.github.com/users/333fred/gists{/gist_id}","starred_url":"https://api.github.com/users/333fred/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/333fred/subscriptions","organizations_url":"https://api.github.com/users/333fred/orgs","repos_url":"https://api.github.com/users/333fred/repos","events_url":"https://api.github.com/users/333fred/events{/privacy}","received_events_url":"https://api.github.com/users/333fred/received_events","type":"User","site_admin":false},"created_at":"2016-12-22T02:06:10Z","updated_at":"2016-12-22T02:06:10Z","body":"Once transitive refs are supported by migration, we'll no longer need to go this roundabout method of removing the files, and we'll be able to just remove them after migration. Closing this."}}
{"actor":{"display_login":"333fred"}}