-
-
Notifications
You must be signed in to change notification settings - Fork 109
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Support loading scripts from embedded resources #30
Comments
Hi, I don't have feedback where have files in a script folder is a problem either on Windows or Linux. More over hide the scripts in the assembly goes against the simplicity and the transparency goal of Evolve IMO. |
My cases:
Simply put I think it makes it easier to deploy. Also flyway supports loading migration scripts from the jar. |
I think this is a good option and would have avoided my issues in #76 |
Hi everyboby, It should be cool to have this feature. We have a WPF application with SQLite, this feature should help us for many way. Regards. |
@lecaillon Is this something you would accept a PR on? If so, I could prioritize implementing it against the current version |
This feature is exencial when you build an application that will run on client. If you expose the spl you have a major security breach. |
+1. I totally agree this is a useful feature. @lecaillon @mhelleborg if any help is needed to move this forward towards a release let me know, it's been almost a year since the PR was opened. |
Let me release the Evolve 2.0 version. |
@GeirGrusom @ld0614 @jouana @modernist @CaueArua Actively working on it for the next release: Evolve 2.1.0 I have to implement it in Evolve, but also in Evolve.MSBuild and the CLI. Usage
Feel free to give me your feedback |
@lecaillon You could improve upon my original interface with a list of assembly / locations, instead of the single EmbeddedResourceContext. I think that would be better than overloading the file locations, and would support potentially loading scripts from multiple assemblies. It also makes the interface contract less ambigous :) |
@mhelleborg You think in this case something like
|
@lecaillon I was more thinking sending in a list of assemblies with a set of locations each. More or less a tuple of assembly and list of locations. Alternatively, just let the user provide a lambda which returns an IEnumerable of migrations. Then they can be loaded from anywhere |
@mhelleborg I need to find a structure easy to use via json or web.config/app.config files for Evolve.MSBuild, or via command line options for Evolve.CLI Thus it implies you cannot filter migrations script by assembly. It is a general filter. That's why at first I thought to allow only one Assembly to avoid confusion |
@mhelleborg I think I will going to keep the
In Evolve.MSBuild or Evolve.CLI those 2 properties will be a list of string where values will be separated by Everyone's ok with that ? |
@lecaillon I think I would prefer having a cleaner API, and maybe drop the functionality from the CLI. I think most of the value proposition for me is using Evolve from code, but other people might have other use cases :) |
Or...
;) |
Add support for using scripts directly as embedded resources.
This simplifies some use cases and makes sure the script can be referenced the same way in all environments.
The text was updated successfully, but these errors were encountered: