-
Notifications
You must be signed in to change notification settings - Fork 59
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
Enhancement:Build and deploy identified artifact #52
Comments
You have the option to use git if you don't use artifact: https://github.com/jalogut/magento2-deployer-plus/blob/master/deploy.php.sample_2_2#L14-L15 Check it out |
Sorry @osrecio, i didn't answer fast enough. Of course i know... Sorry, it seems it was not clear :
The only "issue" is that the "artifact-build" task actually just packages current local version of the (supposed) repository. It is not capable to build a specific version. To be more precise, the artifact task DOESNT use git at all : it packs the local folder, whatever is in it and thats all. Thats pretty dangerous... My suggestion is to rewrite artifact:build to base on update_code : I'll try to share what i've done. Not that simple to create PR as i refactored lot of stuff (version-folder stuff etc...). I'll be back with it this week-end |
I reopened the issue to give you an answer. During this day I will answer you :) |
Done, created PR #58. Hope that will help. |
Well, actually i'm not big fan of "building" the app on the target server : The "build / deploy-artifact" approach makes more sense to me.
Only issue is that the build actually packages the current application version (local).
Actually the standard Deployer "deploy" can handle branch and tag options...
I think that would make sense to enhance the "build" task to grab (git clone) a specific branch / tag version and package it. Then the "deploy-package" would accept an archive parameter to deploy this specific archive.
This way a tagged version artifact could be deployed on staging server, then, exactly same archive could be deployed to production.
Deploying a developper snapshot version archive (what the tools do right now) would just be a particular scenario of the "archive" deployment mecanism.
The text was updated successfully, but these errors were encountered: