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

php 7.4 PR #8534 was merged to master but never merged into any of the 3.0 releases. #10392

Closed
softwareok1 opened this issue Jul 16, 2020 · 5 comments · Fixed by swagger-api/swagger-codegen-generators#1119
Assignees

Comments

@softwareok1
Copy link

softwareok1 commented Jul 16, 2020

Description

If you generate a php-client with codegen 3.0.x composer install fails because "friendsofphp/php-cs-fixer": "~1.12"

Swagger-codegen version

3.0.19

Swagger declaration file content or url
Command line used for generation
Steps to reproduce

generate any openapi3 client with swagger-codegen-cli-3.0.19 and then try composer install

Related issues/PRs

#8533 8533

Suggest a fix/enhancement

merge PR #8534 in releases 3.0.x

@softwareok1 softwareok1 changed the title php PR #8534 was merged to master but never merged into any of the 3.0 releases. php 7.4 PR #8534 was merged to master but never merged into any of the 3.0 releases. Jul 16, 2020
@mmihalev
Copy link

Any updates on this?

@mylemans
Copy link

Ran into a similar issue with PHP8

@JonathanSchndr
Copy link

JonathanSchndr commented Mar 15, 2022

still...

v2

v3

why?

Edit: @softwareok1

The #8533 was fixed on an old state: modules/swagger-codegen/src/main/resources/php/composer.mustache
the new is modules/swagger-codegen/src/main/resources/php-symfony/composer.mustache

@pblan
Copy link

pblan commented Mar 28, 2022

Any fixes yet?

@HugoMario
Copy link
Contributor

This has been fixed by swagger-api/swagger-codegen-generators#1119

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 a pull request may close this issue.

6 participants