search

awferreira / c5-db-migration

Automatically exported from code.google.com/p/c5-db-migration
0 stars 0 forks source link

Use full file name as migration's ID #38

Open GoogleCodeExporter opened 8 years ago

GoogleCodeExporter commented 8 years ago 
The current approach of not using a full name of the migration script
as its ID has some flaws:

- they are actually long and non-descriptive at the same time
  (containing digits only) which makes them an error-prone to share
  (type, spell);

- as everything after the digital prefix simply discarded, any
  meaningful information (for human) had been lost. This makes giving
  a long descriptive names like '4051_add_lastvisit_column.sql' to
  migration scripts pointless.

Another version extractor may be provided which will use the base name
(filename without an extension) of the migration script as its ID.

Original issue reported on code.google.com by mykola.nickishov on 16 Jul 2010 at 10:26

GoogleCodeExporter commented 8 years ago 
I've pushed an initial implementation of base name version extractor to  
http://github.com/manandbytes/c5-db-migration/tree/use-basename-as-id

Feel free to review and comment on it.

BTW, would it be better to provide a simple patch against current trunk?

Original comment by mykola.nickishov on 16 Jul 2010 at 10:33