raml-org / raml-php-parser

Community-maintained RAML parser for php
https://packagist.org/packages/raml-org/raml-php-parser
MIT License
200 stars 77 forks source link

Fixing issue 109 - Allow requests with empty body #114

Closed jefferson-lima closed 6 years ago

jefferson-lima commented 6 years ago

Here's a fix for the problem described in the issue 109.

coveralls commented 6 years ago

Coverage Status

Coverage increased (+0.5%) to 89.655% when pulling 290c76c038ad99305020c1b7df99e1780af7fa6c on jefferson-lima:issue109 into 2f85b8de28fdf3276c9f61fdfdcf6862bca3017c on alecsammon:master.

coveralls commented 6 years ago

Coverage Status

Coverage increased (+0.5%) to 89.655% when pulling 290c76c038ad99305020c1b7df99e1780af7fa6c on jefferson-lima:issue109 into 2f85b8de28fdf3276c9f61fdfdcf6862bca3017c on alecsammon:master.

coveralls commented 6 years ago

Coverage Status

Coverage decreased (-0.06%) to 89.142% when pulling 53588a4ec98c727ea8a086a7be52d38b81e6dac5 on jefferson-lima:issue109 into 2f85b8de28fdf3276c9f61fdfdcf6862bca3017c on alecsammon:master.

coveralls commented 6 years ago

Coverage Status

Coverage decreased (-0.06%) to 89.142% when pulling 53588a4ec98c727ea8a086a7be52d38b81e6dac5 on jefferson-lima:issue109 into 2f85b8de28fdf3276c9f61fdfdcf6862bca3017c on alecsammon:master.

coveralls commented 6 years ago

Coverage Status

Coverage decreased (-0.06%) to 89.142% when pulling 53588a4ec98c727ea8a086a7be52d38b81e6dac5 on jefferson-lima:issue109 into 2f85b8de28fdf3276c9f61fdfdcf6862bca3017c on alecsammon:master.

coveralls commented 6 years ago

Coverage Status

Coverage decreased (-0.06%) to 89.142% when pulling 7961d7c619a7530225b8f8febd323024d4cca721 on jefferson-lima:issue109 into 2f85b8de28fdf3276c9f61fdfdcf6862bca3017c on alecsammon:master.

coveralls commented 6 years ago

Coverage Status

Coverage decreased (-0.06%) to 89.142% when pulling 7961d7c619a7530225b8f8febd323024d4cca721 on jefferson-lima:issue109 into 2f85b8de28fdf3276c9f61fdfdcf6862bca3017c on alecsammon:master.

coveralls commented 6 years ago

Coverage Status

Coverage decreased (-0.06%) to 89.142% when pulling 7961d7c619a7530225b8f8febd323024d4cca721 on jefferson-lima:issue109 into 2f85b8de28fdf3276c9f61fdfdcf6862bca3017c on alecsammon:master.

jefferson-lima commented 6 years ago

Thanks for the review, I will be making the corrections soon.

martin-georgiev commented 6 years ago

@jefferson-lima would you be able to address the code review changes? There is a momentum for moving the repository to RAML.org and it will be nice to clean up the pile of MR before that. Thank you!

jefferson-lima commented 6 years ago

Hi @martin-georgiev, I believe I fixed the issues you pointed out. Would you please give it another look? Note that I squashed my previous commits into a single one to don't mess up the history, I hope this is not a problem.