{"action":"created","issue":{"url":"https://api.github.com/repos/cargomedia/cm/issues/2403","repository_url":"https://api.github.com/repos/cargomedia/cm","labels_url":"https://api.github.com/repos/cargomedia/cm/issues/2403/labels{/name}","comments_url":"https://api.github.com/repos/cargomedia/cm/issues/2403/comments","events_url":"https://api.github.com/repos/cargomedia/cm/issues/2403/events","html_url":"https://github.com/cargomedia/cm/issues/2403","id":186525363,"number":2403,"title":"Allow to include \"description\" in migration file name","user":{"login":"njam","id":360800,"avatar_url":"https://avatars.githubusercontent.com/u/360800?v=3","gravatar_id":"","url":"https://api.github.com/users/njam","html_url":"https://github.com/njam","followers_url":"https://api.github.com/users/njam/followers","following_url":"https://api.github.com/users/njam/following{/other_user}","gists_url":"https://api.github.com/users/njam/gists{/gist_id}","starred_url":"https://api.github.com/users/njam/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/njam/subscriptions","organizations_url":"https://api.github.com/users/njam/orgs","repos_url":"https://api.github.com/users/njam/repos","events_url":"https://api.github.com/users/njam/events{/privacy}","received_events_url":"https://api.github.com/users/njam/received_events","type":"User","site_admin":false},"labels":[{"id":73329293,"url":"https://api.github.com/repos/cargomedia/cm/labels/ready","name":"ready","color":"ededed","default":false},{"id":103886172,"url":"https://api.github.com/repos/cargomedia/cm/labels/slack","name":"slack","color":"009800","default":false}],"state":"open","locked":false,"assignee":{"login":"zazabe","id":401598,"avatar_url":"https://avatars.githubusercontent.com/u/401598?v=3","gravatar_id":"","url":"https://api.github.com/users/zazabe","html_url":"https://github.com/zazabe","followers_url":"https://api.github.com/users/zazabe/followers","following_url":"https://api.github.com/users/zazabe/following{/other_user}","gists_url":"https://api.github.com/users/zazabe/gists{/gist_id}","starred_url":"https://api.github.com/users/zazabe/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/zazabe/subscriptions","organizations_url":"https://api.github.com/users/zazabe/orgs","repos_url":"https://api.github.com/users/zazabe/repos","events_url":"https://api.github.com/users/zazabe/events{/privacy}","received_events_url":"https://api.github.com/users/zazabe/received_events","type":"User","site_admin":false},"assignees":[{"login":"zazabe","id":401598,"avatar_url":"https://avatars.githubusercontent.com/u/401598?v=3","gravatar_id":"","url":"https://api.github.com/users/zazabe","html_url":"https://github.com/zazabe","followers_url":"https://api.github.com/users/zazabe/followers","following_url":"https://api.github.com/users/zazabe/following{/other_user}","gists_url":"https://api.github.com/users/zazabe/gists{/gist_id}","starred_url":"https://api.github.com/users/zazabe/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/zazabe/subscriptions","organizations_url":"https://api.github.com/users/zazabe/orgs","repos_url":"https://api.github.com/users/zazabe/repos","events_url":"https://api.github.com/users/zazabe/events{/privacy}","received_events_url":"https://api.github.com/users/zazabe/received_events","type":"User","site_admin":false}],"milestone":null,"comments":1,"created_at":"2016-11-01T12:44:32Z","updated_at":"2016-12-27T11:13:33Z","closed_at":null,"body":"Goal: allow to include a *description* in the name of a migration script (\"34-some-change\" instead of \"34\"), to avoid VCS conflicts.\r\n\r\nThis was discussed before in https://github.com/cargomedia/cm/issues/588\r\n\r\nThe new migration script could extend `CM_Provision_Script_Abstract` and use `CM_Provision_Script_OptionBased` for persisting whether a script has ran or not.\r\n-> Or it could use a separate table, which can be used in more powerful ways in the future (keep timestamps etc).\r\n\r\nThe filename should still include a timestamp for ordering. For example the file could be named `1478003870-some-change.php` and the class `CM_Provision_Migration_1478003870-some-change`.\r\n\r\nWe should introduce a CLI helper to create a new migration. The *description* could by default be taken from the GIT branch name.\r\n\r\ncc @tomaszdurka @dlondero @fauvel"},"comment":{"url":"https://api.github.com/repos/cargomedia/cm/issues/comments/269312294","html_url":"https://github.com/cargomedia/cm/issues/2403#issuecomment-269312294","issue_url":"https://api.github.com/repos/cargomedia/cm/issues/2403","id":269312294,"user":{"login":"zazabe","id":401598,"avatar_url":"https://avatars.githubusercontent.com/u/401598?v=3","gravatar_id":"","url":"https://api.github.com/users/zazabe","html_url":"https://github.com/zazabe","followers_url":"https://api.github.com/users/zazabe/followers","following_url":"https://api.github.com/users/zazabe/following{/other_user}","gists_url":"https://api.github.com/users/zazabe/gists{/gist_id}","starred_url":"https://api.github.com/users/zazabe/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/zazabe/subscriptions","organizations_url":"https://api.github.com/users/zazabe/orgs","repos_url":"https://api.github.com/users/zazabe/repos","events_url":"https://api.github.com/users/zazabe/events{/privacy}","received_events_url":"https://api.github.com/users/zazabe/received_events","type":"User","site_admin":false},"created_at":"2016-12-27T11:13:33Z","updated_at":"2016-12-27T11:13:33Z","body":"- Let's use a separate table, maybe auto-generated like described [here](https://github.com/cargomedia/cm/issues/588#issuecomment-78231730)\r\n- I would suggest to always generate the migration class, with a reference to the previous migration script.  \r\n  Then, we could add a CI script to test if there's multiple migration classes with a ref to the same migration script.\r\n\r\n```\r\nCM_Provision_MigrationScript extends CM_Model_Abstract\r\n  - filename: string\r\n  - classname: string\r\n  - previous: CM_Provision_MigrationScript|null \r\n  - execStamp: DateTime  \r\n```\r\n\r\nwdyt?"}}
{"actor":{"display_login":"zazabe"}}