-
Notifications
You must be signed in to change notification settings - Fork 1
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
More API Blueprint fixes #2
More API Blueprint fixes #2
Conversation
a0baadf
to
bb5546d
Compare
@@ -16,34 +19,49 @@ FORMAT: 1A | |||
## {{ route_name }} [{{ route }}] | |||
{{# description }} | |||
|
|||
description: {{ description }} | |||
{{ description }} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not sure what value this should have... or if it even works.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Cool, thanks, we just need to add apiblueprint validations in spec suite, and we'll be done.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed the specs, and added an expectation for the "route explanations". I can't seem to write an expectation for the "method explanations", although it works when I use the gem. Something seems off with the http_method
s in the specs for some reason.
{{# attributes }} | ||
+ {{ name }}{{# example }}: {{ example }}{{/ example }}{{# properties_description }} ({{ properties_description }}){{/ properties_description }}{{# description }} - {{ description }}{{/ description }} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
example
is the RSpec example in this case, which we don't want. I wanted to render the actual value of the param here, but I don't think it's possible without some serious refactoring. I did add the ability to define the "default" though.
Anyhow, if you have an idea how we could get the actual parameter value in here that would be amazing 🙏
Hi 👋
Thanks for fixing the API Blueprint stuff.
Here are some further improvements I made (possibly):
route
s and methods (eg.get ...
)Does this fix the issue you have with attributes?