-
Notifications
You must be signed in to change notification settings - Fork 418
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 syntax error #116 #121
Comments
Ping :) |
Hey @ifrolikov thanks for your ping, i had missed it this one. i'm going to check it. |
@ifrolikov can you create the PR from your fork so we can merge your changes here? |
Hey @HugoMario , #123, I create PR :) |
Hey, @HugoMario! I'm sorry, that I write really often, but it is important to me. Swagger codegen get an old snapshot of swagger-codegen-generators while building: |
It's ok @ifrolikov, feel free to ping any time you need. it should look for latest, let me check this and let you know. |
@ifrolikov what command are you using for building, would you try with:
in case you're using a different one? |
I'm build image from dockerfile: I'm look repo https://oss.sonatype.org/content/repositories/snapshots/io/swagger/swagger-codegen-generators/1.0.0-SNAPSHOT/ and latest version of snapshot is 2018-07-20. Should not a new version of the snapshot be created after my PR merged? (#126) |
I think this is related to the circular dependency issue we have in We need to trigger a build on swagger-codegen in order to fix this. This circular dependency issue should be fixed with project refactoring tasks that we're working right now. I'll try to manually fix this temporary and let you know. |
I forgot about it, thanks! |
Hey! I create new issue, because previous issue was closed.
There are {{#hasMore}} attribute was still in some places:
Thank!
The text was updated successfully, but these errors were encountered: