mtrajano / laravel-swagger

Auto generates the swagger documentation of a laravel project based on best practices and simple assumptions
167 stars 71 forks source link

Laravel 7 support #43

Closed nenads closed 4 years ago

nenads commented 4 years ago

Description:

composer require mtrajano/laravel-swagger

Problem 1

mtrajano commented 4 years ago

Hey I will look into this tonight, you're the 2nd person to report this. Looks like laravel 7 or some package it's using requires version 5 of phpdocumentor/reflection-docblock and we're still on 4.3. I aready upgraded on the 7.x branch that I'm cleaning up and there were no breaking changes.

nenads commented 4 years ago

Grate news !!!

Thanks !!!

On Mon, Apr 20, 2020, 16:21 Mauricio Trajano notifications@github.com wrote:

Hey I will look into this tonight, you're the 2nd person to report this. Looks like laravel 7 or some package it's using requires version 5 of phpdocumentor/reflection-docblock and we're still on 4.3. I aready upgraded on the 7.x branch that I'm cleaning up and there were no breaking changes.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/mtrajano/laravel-swagger/issues/43#issuecomment-616586103, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAN5LMFZAOWXN3UNOLJC4M3RNRK7BANCNFSM4MMOIWNA .

zFly commented 4 years ago

@mtrajano This bug also affects

I tried composer require mtrajano/laravel-swagger:0.7.x-dev but I got the same error. (because you mentioned the 7.x branch)

Just wanted you to know this. Keep up the great work!! :)

mtrajano commented 4 years ago

@zFly I had not pushed my changes yet. I just updated the v0.7 branch would you or @nenads be able to try again? Sorry about that

mtrajano commented 4 years ago

I probably won't push this to the the v0.6.x branch since it breaks compatibility, can you please try upgrating to 0.7.x-dev @nenads? There are a lot of extra exciting features coming 😄

zFly commented 4 years ago

Using composer require mtrajano/laravel-swagger:0.7.x-dev worked for me.

Thank you for the quick response :)

nenads commented 4 years ago

Same !!!! 👍

Using composer require mtrajano/laravel-swagger:0.7.x-dev worked for me.

Thank you for the quick response :)

mtrajano commented 4 years ago

Great!

zeke8402 commented 4 years ago

I get this when trying to install the 0.7.x-dev branch using PHP 7.3 and PHP 7.4 (tried both as a sanity check), fresh Laravel install.

Screen Shot 2020-07-17 at 00 47 30 AM
tim-rosseel commented 3 years ago

Same as https://github.com/mtrajano/laravel-swagger/issues/52?

I tried composer require mtrajano/laravel-swagger:0.7.x-dev, but that asked for a token that I don't know...