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

[payum] update the bundle composer.json #807

Merged
merged 1 commit into from
Jan 10, 2014

Conversation

makasim
Copy link
Contributor

@makasim makasim commented Jan 9, 2014

while trying to fix Sylius/Sylius-Standard#22 I discovered that sylius payum bundle has outdated requirements. This PR is a blocker for Sylius/Sylius-Standard#22 issue.

@makasim
Copy link
Contributor Author

makasim commented Jan 9, 2014

@pjedrzejewski ping

@Richtermeister
Copy link
Contributor

Also currently blocked by this 😓

@kayue
Copy link
Contributor

kayue commented Jan 10, 2014

@Richtermeister In my case adding --dev to composer help, maybe you should try it too.

@makasim
Copy link
Contributor Author

makasim commented Jan 10, 2014

In my case adding --dev to composer help, maybe you should try it too.

hm, it must not help

@stloyd
Copy link
Contributor

stloyd commented Jan 10, 2014

@kayue If you need to add --dev, then you are using quite old Composer version, as in "latest" it was changed, and require-dev are added automatically when calling install or update.

@makasim This is partial fix I would say, mostly because in Sylius-Standard there are still old references: https://github.com/Sylius/Sylius-Standard/blob/master/composer.json#L40-L43

stloyd added a commit that referenced this pull request Jan 10, 2014
[payum] update the bundle composer.json
@stloyd stloyd merged commit 2715a63 into Sylius:master Jan 10, 2014
@makasim
Copy link
Contributor Author

makasim commented Jan 10, 2014

@stloyd I cannot updated composer.lock of sylius-standard. because sylius payum bundle on packagist still requires 0.6. That's why I wrote it blocked me. Thanks for merging!

@makasim makasim deleted the payum-upgrade-to-0.7 branch January 10, 2014 08:31
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.

Getting a fatal error when updating the bundles
4 participants