search

phoenixnap / springmvc-raml-plugin

Spring MVC - RAML Spec Synchroniser Plugin. A Maven plugin designed to Generate Server & Client code in Spring from a RAML API descriptor and conversely, a RAML API document from the SpringMVC Server implementation.
Apache License 2.0
136 stars 84 forks source link

Issue 211 #216

Closed yuranos closed 6 years ago

yuranos commented 6 years ago

Based on #211 @stojsavljevic , please review.

stojsavljevic commented 6 years ago

@yuranos in #211 I stated that this is a feature not a bug :) So what's the point of this PR?

yuranos commented 6 years ago

HI, @stojsavljevic ! I created a PR before I saw your comment:) Having said that, I think it's an unnecessary limitation. It's more common to use plural notation, but there's more than one way to skin a cat, i.e., a lot of people will argue otherwise. My suggestion is not breaking anything that was already there, it's just making the decision based on something that is less debatable than naming convention(id in the path). Do you see any potential problems with my idea? Please, check for more evidence that the view is not one-sided: https://stackoverflow.com/questions/6845772/rest-uri-convention-singular-or-plural-name-of-resource-while-creating-it https://stackoverflow.com/questions/778203/are-there-any-naming-convention-guidelines-for-rest-apis People have different views on pluralism vs singularity.

yuranos commented 6 years ago

@stojsavljevic , time to get back to this one.