Add ability to filter routes to a certain prefix

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

Add ability to filter routes to a certain prefix #2

Closed overint closed 6 years ago

overint commented 6 years ago

I have a bunch of non-api routes in my app, which I don't need docs generated for. This PR adds a --filter option which you can use to filter the generation to a certain prefix.

Let me know if there are any issues or changes you would like.

mtrajano commented 6 years ago

@overint Hey thanks for the contribution! I had something similar on my TODO. Would you mind just writing a couple of tests to complement your changes? In any case it looks good to me.

I did merge your other pr though.

overint commented 6 years ago

No worries, will get some tests written up and update the PR.

Thanks!

overint commented 6 years ago

I've added tests and updated the readme, let me know if anything else is needed, thanks! (may I suggest setting up travis to run the tests (and possibly phpcs))

overint commented 6 years ago

Thanks for merging. If you need any help with other feature let me know 👍