You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@camel.apache.org by GitBox <gi...@apache.org> on 2020/09/28 19:52:48 UTC

[GitHub] [camel] santam85 opened a new pull request #4311: [CAMEL-15588] Updated swagger-core to 1.6.2

santam85 opened a new pull request #4311:
URL: https://github.com/apache/camel/pull/4311


   Updated swagger-core to 1.6.2
   Updated swagger-parser to 1.0.51
   
   This is to address a NoSuchMethod exception when upgrading to latest jackson. See https://github.com/swagger-api/swagger-core/issues/3554 for details.


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [camel] santam85 commented on pull request #4311: [CAMEL-15588] Updated swagger-core to 1.6.2

Posted by GitBox <gi...@apache.org>.
santam85 commented on pull request #4311:
URL: https://github.com/apache/camel/pull/4311#issuecomment-700708134


   > Lgtm. Please can you double check the swagger karaf feature in the repo Apache/camel-karaf? You'll need to fully build the camel repo and then run the build of camel-karaf. You can check the result under platform/karaf/features of the camel-karaf repository, by running an mvn verify or install. Thanks
   
   Turns out I did have to update jackson-core in order to satisfy the updated swagger dependency requirements on Karaf.
   There seems to be files generated during the full CI build that I can't seem to generate locally, is that expected? 


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [camel] oscerd commented on pull request #4311: [CAMEL-15588] Updated swagger-core to 1.6.2

Posted by GitBox <gi...@apache.org>.
oscerd commented on pull request #4311:
URL: https://github.com/apache/camel/pull/4311#issuecomment-700710075


   I'll have a look before merging, thanks.


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [camel] santam85 commented on pull request #4311: [CAMEL-15588] Updated swagger-core to 1.6.2

Posted by GitBox <gi...@apache.org>.
santam85 commented on pull request #4311:
URL: https://github.com/apache/camel/pull/4311#issuecomment-700812658


   > I think you need to do git pull --rebase origin master, in this way your commit will be HEAD
   
   Sorry, I messed this up in my IDE


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [camel] oscerd commented on pull request #4311: [CAMEL-15588] Updated swagger-core to 1.6.2

Posted by GitBox <gi...@apache.org>.
oscerd commented on pull request #4311:
URL: https://github.com/apache/camel/pull/4311#issuecomment-700795433


   I think you need to do git pull --rebase origin master, in this way your commit will be HEAD


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [camel] oscerd commented on pull request #4311: [CAMEL-15588] Updated swagger-core to 1.6.2

Posted by GitBox <gi...@apache.org>.
oscerd commented on pull request #4311:
URL: https://github.com/apache/camel/pull/4311#issuecomment-701177666


   Merged on master. Thanks.


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [camel] oscerd closed pull request #4311: [CAMEL-15588] Updated swagger-core to 1.6.2

Posted by GitBox <gi...@apache.org>.
oscerd closed pull request #4311:
URL: https://github.com/apache/camel/pull/4311


   


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org