{"comment":{"url":"https://api.github.com/repos/qgis/QGIS/comments/17727488","html_url":"https://github.com/qgis/QGIS/commit/87121d63a8ca28dc75631d3ef6a59c025388a026#commitcomment-17727488","id":17727488,"user":{"login":"m-kuhn","id":588407,"avatar_url":"https://avatars.githubusercontent.com/u/588407?v=3","gravatar_id":"","url":"https://api.github.com/users/m-kuhn","html_url":"https://github.com/m-kuhn","followers_url":"https://api.github.com/users/m-kuhn/followers","following_url":"https://api.github.com/users/m-kuhn/following{/other_user}","gists_url":"https://api.github.com/users/m-kuhn/gists{/gist_id}","starred_url":"https://api.github.com/users/m-kuhn/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/m-kuhn/subscriptions","organizations_url":"https://api.github.com/users/m-kuhn/orgs","repos_url":"https://api.github.com/users/m-kuhn/repos","events_url":"https://api.github.com/users/m-kuhn/events{/privacy}","received_events_url":"https://api.github.com/users/m-kuhn/received_events","type":"User","site_admin":false},"position":22,"line":22,"path":"src/core/qgsbillboardregistry.cpp","commit_id":"87121d63a8ca28dc75631d3ef6a59c025388a026","created_at":"2016-06-03T08:16:11Z","updated_at":"2016-06-03T08:16:11Z","body":"Or we allow registering plugins their custom API which would IMO be cleaner but needs to be discussed.\r\n\r\n> Sure, if you feel uncomfortable with this API I have no problems removing it again for the time being.\r\n\r\nI would say so.\r\n\r\nIn general it's good practice to create multiple pull requests for independent functionality so it can be reviewed and merged individually.\r\nAnd on a related note, it is also good practice to give credits when you make a pull request or presentation heavily based on someone else work."}}
{"actor":{"display_login":"m-kuhn"}}