search

migrating-ravens / RavenMigrations

A small migrations framework to help you manage your RavenDB Instance.
MIT License
53 stars 24 forks source link

Throw when migrations are missing attribute. #32

Closed dportzline83 closed 9 years ago

dportzline83 commented 9 years ago

Throw an exception when migrations are missing the MigrationAttribute. If the class does not have a parameterless constructor or it is abstract, exclude it from the migrations altogether. Addresses #31.

rogersillito commented 9 years ago

I've not run your new code but the rationale looks sound, and if it prevents a scenario where a migration is silently skipped while still supporting sub-classing, it sounds like a win-win.