{"action":"created","issue":{"url":"https://api.github.com/repos/ideatosrl/Idephix/issues/71","repository_url":"https://api.github.com/repos/ideatosrl/Idephix","labels_url":"https://api.github.com/repos/ideatosrl/Idephix/issues/71/labels{/name}","comments_url":"https://api.github.com/repos/ideatosrl/Idephix/issues/71/comments","events_url":"https://api.github.com/repos/ideatosrl/Idephix/issues/71/events","html_url":"https://github.com/ideatosrl/Idephix/pull/71","id":165971853,"number":71,"title":"Extension refactoring","user":{"login":"ftassi","id":176622,"avatar_url":"https://avatars.githubusercontent.com/u/176622?v=3","gravatar_id":"","url":"https://api.github.com/users/ftassi","html_url":"https://github.com/ftassi","followers_url":"https://api.github.com/users/ftassi/followers","following_url":"https://api.github.com/users/ftassi/following{/other_user}","gists_url":"https://api.github.com/users/ftassi/gists{/gist_id}","starred_url":"https://api.github.com/users/ftassi/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ftassi/subscriptions","organizations_url":"https://api.github.com/users/ftassi/orgs","repos_url":"https://api.github.com/users/ftassi/repos","events_url":"https://api.github.com/users/ftassi/events{/privacy}","received_events_url":"https://api.github.com/users/ftassi/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/ideatosrl/Idephix/labels/in%20progress","name":"in progress","color":"c5def5"}],"state":"open","locked":false,"assignee":{"login":"ftassi","id":176622,"avatar_url":"https://avatars.githubusercontent.com/u/176622?v=3","gravatar_id":"","url":"https://api.github.com/users/ftassi","html_url":"https://github.com/ftassi","followers_url":"https://api.github.com/users/ftassi/followers","following_url":"https://api.github.com/users/ftassi/following{/other_user}","gists_url":"https://api.github.com/users/ftassi/gists{/gist_id}","starred_url":"https://api.github.com/users/ftassi/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ftassi/subscriptions","organizations_url":"https://api.github.com/users/ftassi/orgs","repos_url":"https://api.github.com/users/ftassi/repos","events_url":"https://api.github.com/users/ftassi/events{/privacy}","received_events_url":"https://api.github.com/users/ftassi/received_events","type":"User","site_admin":false},"assignees":[{"login":"ftassi","id":176622,"avatar_url":"https://avatars.githubusercontent.com/u/176622?v=3","gravatar_id":"","url":"https://api.github.com/users/ftassi","html_url":"https://github.com/ftassi","followers_url":"https://api.github.com/users/ftassi/followers","following_url":"https://api.github.com/users/ftassi/following{/other_user}","gists_url":"https://api.github.com/users/ftassi/gists{/gist_id}","starred_url":"https://api.github.com/users/ftassi/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ftassi/subscriptions","organizations_url":"https://api.github.com/users/ftassi/orgs","repos_url":"https://api.github.com/users/ftassi/repos","events_url":"https://api.github.com/users/ftassi/events{/privacy}","received_events_url":"https://api.github.com/users/ftassi/received_events","type":"User","site_admin":false}],"milestone":{"url":"https://api.github.com/repos/ideatosrl/Idephix/milestones/1","html_url":"https://github.com/ideatosrl/Idephix/milestones/1.0","labels_url":"https://api.github.com/repos/ideatosrl/Idephix/milestones/1/labels","id":1136732,"number":1,"title":"1.0","description":"","creator":{"login":"micheleorselli","id":173605,"avatar_url":"https://avatars.githubusercontent.com/u/173605?v=3","gravatar_id":"","url":"https://api.github.com/users/micheleorselli","html_url":"https://github.com/micheleorselli","followers_url":"https://api.github.com/users/micheleorselli/followers","following_url":"https://api.github.com/users/micheleorselli/following{/other_user}","gists_url":"https://api.github.com/users/micheleorselli/gists{/gist_id}","starred_url":"https://api.github.com/users/micheleorselli/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/micheleorselli/subscriptions","organizations_url":"https://api.github.com/users/micheleorselli/orgs","repos_url":"https://api.github.com/users/micheleorselli/repos","events_url":"https://api.github.com/users/micheleorselli/events{/privacy}","received_events_url":"https://api.github.com/users/micheleorselli/received_events","type":"User","site_admin":false},"open_issues":8,"closed_issues":4,"state":"open","created_at":"2015-05-27T19:45:56Z","updated_at":"2016-07-23T11:11:49Z","due_on":"2016-09-29T22:00:00Z","closed_at":null},"comments":5,"created_at":"2016-07-17T11:06:25Z","updated_at":"2016-07-27T07:43:36Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/ideatosrl/Idephix/pulls/71","html_url":"https://github.com/ideatosrl/Idephix/pull/71","diff_url":"https://github.com/ideatosrl/Idephix/pull/71.diff","patch_url":"https://github.com/ideatosrl/Idephix/pull/71.patch"},"body":"Ho riscritto un po' come vengono gestite le estensioni. Questa forma mi pare flessibile abbastanza da poterla tenere pubblica per un po'. \r\n\r\nRiscrivendo l'estensione per il deploy ho sentito la necessità di avere un modo per sovrascrivere alcune parti (la creazione dell'artefatto imho è troppo specifica del progetto per metterla in un' estensione, mentre le fasi precedenti e successive al deploy sono sempre le stesse). Così ho provato ad ipotizzare un'implementazione che permetta in qualche modo di fare override. Non mi convince molto perché non sono sicuro che sia chiarissimo il funzionamento[1], ma vorrei provarlo per riscrivere il deploy prima di bocciarlo, a meno che non abbiate suggerimenti forti al riguardo. \r\n\r\n[1] Il problema è che definendo un task che si chiama come un metodo di un'estensione faccio in modo che venga eseguito il mio codice al posto di quello dell'estensione stessa, solo che sto sostituendo un metodo, che quindi ha accesso all'istanza dell'estensione, con una funziona che ha uno scope completamente diverso. Potrebbe non essere un problema, dato che se sto facendo un override voglio fare delle cose (e so cosa devo fare), ma il tutto mi sembra un po' farraginoso.\r\n\r\n<!---\r\n@huboard:{\"order\":71.0,\"milestone_order\":71,\"custom_state\":\"\"}\n-->"},"comment":{"url":"https://api.github.com/repos/ideatosrl/Idephix/issues/comments/235510785","html_url":"https://github.com/ideatosrl/Idephix/pull/71#issuecomment-235510785","issue_url":"https://api.github.com/repos/ideatosrl/Idephix/issues/71","id":235510785,"user":{"login":"micheleorselli","id":173605,"avatar_url":"https://avatars.githubusercontent.com/u/173605?v=3","gravatar_id":"","url":"https://api.github.com/users/micheleorselli","html_url":"https://github.com/micheleorselli","followers_url":"https://api.github.com/users/micheleorselli/followers","following_url":"https://api.github.com/users/micheleorselli/following{/other_user}","gists_url":"https://api.github.com/users/micheleorselli/gists{/gist_id}","starred_url":"https://api.github.com/users/micheleorselli/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/micheleorselli/subscriptions","organizations_url":"https://api.github.com/users/micheleorselli/orgs","repos_url":"https://api.github.com/users/micheleorselli/repos","events_url":"https://api.github.com/users/micheleorselli/events{/privacy}","received_events_url":"https://api.github.com/users/micheleorselli/received_events","type":"User","site_admin":false},"created_at":"2016-07-27T07:43:36Z","updated_at":"2016-07-27T07:43:36Z","body":"rispondo ai tuoi punti\r\n1) i phar di fatto sono file compressi, non penso che si possa dire \"questo file non lo comprimere\". Per risolvere il problema della formattazione forse potremmo trattare tutto come una mega stringa e copiarla nell'idxfile? \r\n2) stiamo parlando di 10/15/20/100 file? mi sembra un numero esiguo rispetto ai vendor che abbiamo e che vengono inclusi nel phar. Sicuramente la possibilità di scaricarle da qualche parte è interessante al momento penso possa bastare mettere tutto nel phar"}}
{"actor":{"display_login":"micheleorselli"}}