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

Merge service libraries into one repository and use git tree split #93

Merged
merged 554 commits into from
Dec 3, 2013

Conversation

makasim
Copy link
Member

@makasim makasim commented Dec 1, 2013

IMO it starts to be annoying that every payment has own repository & is not any way to get all of them in one time, also most of those implementation are really similar when you compare code bases...

What I'm suggesting is a merge into main repo all of those (could be hard to maintain at some point), or create new repository, i.e. PayumGateway (or something) and merge all those currently separated repositories (AuthorizeNetAim, Be2Bill etc.).

With this approch you could have in main catalog an composer.json file:

{
    "name": "payum/gateways",
    "type": "library",
    "license": "MIT",
    "require": {
        "php": ">=5.3.2",
        "payum/payum": "0.7.x-dev",
    },
    "autoload": {
        "psr-0": { "Payum\\Gateway\\": "src/" }
    }
}

All gateways would be store i.e. Payum\Gateway\Be2Bill, and on that folder it will be same as actual content of repository with git tree split.

The approach is same as used by Symfony or Sylius.

…420f052d22eb3431cb'

git-subtree-dir: src/Payum/AuthorizeNet/Aim
git-subtree-mainline: f13af78
git-subtree-split: 45e3b29
…463000584b92'

git-subtree-dir: src/Payum/Paypal/Ipn
git-subtree-mainline: d2a2649
git-subtree-split: efaf5a6
makasim added a commit that referenced this pull request Dec 3, 2013
[RFC] Merge service libraries into one repository and use git tree split
@makasim makasim merged commit b8af2fb into master Dec 3, 2013
@makasim makasim deleted the merge-all-repos branch December 3, 2013 14:47
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.

None yet

8 participants