Resource inheritance fix for #23 & #47

jdiegodcp / ramlfications

Python parser for RAML

https://ramlfications.readthedocs.org

Apache License 2.0

234 stars 50 forks source link

Closed econchick closed 8 years ago

econchick commented 8 years ago

I'm hoping this works!

Definitely should have broken this up into better commits.

Generally the fix for resource inheritance is within the while loop in ramlfications/parser.py::create_resource_types function.

Everything else has been pretty much steps towards more DRY and removing idiotic duplication.

Also trying out the dev version numbers for in-between releases.

codecov-io commented 8 years ago

Merging #73 into master will decrease coverage by -0.85% as of d6c8d56

Powered by Codecov. Updated on successful CI builds.

econchick commented 8 years ago

Merging - codecov/project was set to auto-reject for any decrease in coverage; I set a minimum to 90% but the hook didn't update for this page.

Also - super scared to merge this without a proper review but F&CK IT, WE'LL DO IT LIVE.