Skip to content
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

[WIP] method based deployment #50

Open
wants to merge 10 commits into
base: master
Choose a base branch
from
Open

[WIP] method based deployment #50

wants to merge 10 commits into from

Conversation

tamagokun
Copy link
Owner

This is a rather large re-vamp of how deployment stacks are built.

Not ready

Deployments with Pomander have a strict release/no-release structure with some options one of which being scm to help offer a more flexible means to deploy for all projects. Problem is there is no way to break out of this structure and create deployment paths for other things (i.e. Heroku, Docker)

This new way allows that to happen..

All deployment related tasks are driven by a Method object. The method ultimately controls exactly how a deployment works. The base Method class implements the standard release / no-release structure for deploying, rolling back, cleanup, finalize, etc. There are stubbed methods for setup_code and update_code that you can sub class and customize to your needs, or just implement all new methods.

Breaking changes:

This release will introduce some potential breaking changes for anyone that used the $app->env->scm object. That object is now replaced by method.

  • - Git method
  • - Subversion method
  • - Rsync method
  • [ ] - Heroku method (later)
  • - New tests with passing current tests

@tamagokun
Copy link
Owner Author

This will also take care of #31 #47

@tamagokun
Copy link
Owner Author

For Heroku deployments Pomander needs to be able to trigger a heroku task after composer installs all of its deps:

This can probably be achieved by means of a Composer plugin: https://getcomposer.org/doc/articles/plugins.md

After the plugin is loaded, pom heroku

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant