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 syntax error #116 #121

Closed
ifrolikov opened this issue Jul 13, 2018 · 12 comments
Closed

php syntax error #116 #121

ifrolikov opened this issue Jul 13, 2018 · 12 comments
Assignees

Comments

@ifrolikov
Copy link

Hey! I create new issue, because previous issue was closed.
There are {{#hasMore}} attribute was still in some places:

  • src/main/resources/v2/php/README.mustache
  • src/main/resources/v2/php/api.mustache
  • src/main/resources/v2/php/api_doc.mustache
  • src/main/resources/v2/php/model_generic.mustache

Thank!

@ifrolikov
Copy link
Author

@ifrolikov
Copy link
Author

Ping :)

@HugoMario HugoMario self-assigned this Jul 17, 2018
@HugoMario
Copy link
Contributor

HugoMario commented Jul 17, 2018

Hey @ifrolikov thanks for your ping, i had missed it this one.

i'm going to check it.

@HugoMario
Copy link
Contributor

@ifrolikov can you create the PR from your fork so we can merge your changes here?

@ifrolikov
Copy link
Author

Hey @HugoMario , #123, I create PR :)

@ifrolikov
Copy link
Author

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:
image
Is this problem of build, which you told me?

@HugoMario
Copy link
Contributor

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.

@HugoMario
Copy link
Contributor

@ifrolikov what command are you using for building, would you try with:

mvn clean install -U

in case you're using a different one?

@ifrolikov
Copy link
Author

I'm build image from dockerfile:
image.

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)

@HugoMario
Copy link
Contributor

I think this is related to the circular dependency issue we have in swagger-codegen repo for 3.0.0.

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.

@pavog
Copy link

pavog commented Aug 22, 2018

Hey I think you can close this issue now, since #123 and #124 have been merged.

@ifrolikov
Copy link
Author

I forgot about it, thanks!

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

No branches or pull requests

3 participants