{"action":"created","issue":{"url":"https://api.github.com/repos/opsdroid/opsdroid/issues/654","repository_url":"https://api.github.com/repos/opsdroid/opsdroid","labels_url":"https://api.github.com/repos/opsdroid/opsdroid/issues/654/labels{/name}","comments_url":"https://api.github.com/repos/opsdroid/opsdroid/issues/654/comments","events_url":"https://api.github.com/repos/opsdroid/opsdroid/issues/654/events","html_url":"https://github.com/opsdroid/opsdroid/issues/654","id":363661358,"node_id":"MDU6SXNzdWUzNjM2NjEzNTg=","number":654,"title":"Treat skills with explicit path as local","user":{"login":"colonelkrud","id":23349522,"node_id":"MDQ6VXNlcjIzMzQ5NTIy","avatar_url":"https://avatars2.githubusercontent.com/u/23349522?v=4","gravatar_id":"","url":"https://api.github.com/users/colonelkrud","html_url":"https://github.com/colonelkrud","followers_url":"https://api.github.com/users/colonelkrud/followers","following_url":"https://api.github.com/users/colonelkrud/following{/other_user}","gists_url":"https://api.github.com/users/colonelkrud/gists{/gist_id}","starred_url":"https://api.github.com/users/colonelkrud/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/colonelkrud/subscriptions","organizations_url":"https://api.github.com/users/colonelkrud/orgs","repos_url":"https://api.github.com/users/colonelkrud/repos","events_url":"https://api.github.com/users/colonelkrud/events{/privacy}","received_events_url":"https://api.github.com/users/colonelkrud/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":0,"created_at":"2018-09-25T16:19:31Z","updated_at":"2018-09-25T18:07:41Z","closed_at":null,"author_association":"NONE","body":"When defining custom skills in the configuration file, we have the ability to define a path for the skill. When this path is defined, we should be able to safely assume that the skill is local and accessible by the bot and therefore should not need to be cached.\r\n\r\nMy suggestion is to treat skills which have been explicitly defined with a local path as local skills and automatically apply the `nocache=true` unless caching is otherwise defined in the config."},"comment":{"url":"https://api.github.com/repos/opsdroid/opsdroid/issues/comments/424444948","html_url":"https://github.com/opsdroid/opsdroid/issues/654#issuecomment-424444948","issue_url":"https://api.github.com/repos/opsdroid/opsdroid/issues/654","id":424444948,"node_id":"MDEyOklzc3VlQ29tbWVudDQyNDQ0NDk0OA==","user":{"login":"FabioRosado","id":3131401,"node_id":"MDQ6VXNlcjMxMzE0MDE=","avatar_url":"https://avatars0.githubusercontent.com/u/3131401?v=4","gravatar_id":"","url":"https://api.github.com/users/FabioRosado","html_url":"https://github.com/FabioRosado","followers_url":"https://api.github.com/users/FabioRosado/followers","following_url":"https://api.github.com/users/FabioRosado/following{/other_user}","gists_url":"https://api.github.com/users/FabioRosado/gists{/gist_id}","starred_url":"https://api.github.com/users/FabioRosado/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/FabioRosado/subscriptions","organizations_url":"https://api.github.com/users/FabioRosado/orgs","repos_url":"https://api.github.com/users/FabioRosado/repos","events_url":"https://api.github.com/users/FabioRosado/events{/privacy}","received_events_url":"https://api.github.com/users/FabioRosado/received_events","type":"User","site_admin":false},"created_at":"2018-09-25T18:07:41Z","updated_at":"2018-09-25T18:07:41Z","author_association":"CONTRIBUTOR","body":"I like that idea and it does make sense to have the nocache set to true if the path is available in the config when you are developing the skill. \r\n\r\nWe need to check with the community if this is something we should implement as well, the reason for that being that some devs are probably using their own skills with opsdroid and perhaps having the nocache set could cause a burden to them when opsdroid is run."}}
{"actor":{"display_login":"FabioRosado"}}