-
Notifications
You must be signed in to change notification settings - Fork 77
Conversation
…piDefinition::setBaseUrl
2 similar comments
@InfopactMLoos , Sorry for the late reply. I'll have a look at this hopefully within the next day or so. As for whether this eventually gets merged, I'm unsure as it looks like @alecsammon is rather inactive at the moment. I'll be happy to install directly from your fork, assuming it is tagged correctly and won't go away anytime soon :) |
@InfopactMLoos can you estimate how much work will be needed to implement the remaining bits from RAML 1.0 specification based on this MR? I think there is a momentum so this can be pushed forward. |
What's the status of this? I'd like to use this library, but I'm only use RAML 1.0. This PR was opened over a year ago, and there's been no update on this at all. |
I don't think anything is being merged at the moment. You should read over this issue: https://github.com/alecsammon/php-raml-parser/issues/117 The eLama fork is more inline with the 1.0 spec than this. |
I have done a first implementation of types and their validations. Please review the changes and create a new branch (v3.0?) for this request, as was discussed in Issue 108. It is a working in progress for the entire 1.0 specification but for my use case it already works. I have made a quick shortlist of things we should add for a complete 1.0 release in the README. Also we can probably use more unit tests.
Also @AnzeDspot, @therealgambo