Enforce "since major.minor.patch" javadoc convention

Closes gh-1345

Author: bclozel

spring-graphql-test/src/main/java/org/springframework/graphql/test/tester/GraphQlTester.java

@@ -152,7 +152,7 @@ interface Request<T extends Request<T>> {
 		 * and is defined with the {@code fragment} keyword.
 		 * @param fragment the fragment definition
 		 * @return this request spec
-		 * @since 1.3
+		 * @since 1.3.0
 		 * @see <a href="http://spec.graphql.org/October2021/#sec-Language.Fragments">Fragments specification</a>
 		 */
 		T fragment(String fragment);
@@ -164,7 +164,7 @@ interface Request<T extends Request<T>> {
 		 * @param fragmentName the name of the fragment to append
 		 * @return this request spec
 		 * @throws IllegalArgumentException if the fragmentName cannot be resolved
-		 * @since 1.3
+		 * @since 1.3.0
 		 */
 		T fragmentName(String fragmentName);
 

spring-graphql/src/main/java/org/springframework/graphql/ResponseError.java

@@ -27,7 +27,7 @@
  * Represents a GraphQL response error.
  *
  * @author Rossen Stoyanchev
- * @since 1.0
+ * @since 1.0.0
  */
 public interface ResponseError {
 

spring-graphql/src/main/java/org/springframework/graphql/data/method/annotation/support/AnnotatedControllerDetectionSupport.java

@@ -221,7 +221,7 @@ public void setExecutor(Executor executor) {
 	 * On Java 20 and lower, the predicate returns false. You can configure the
 	 * predicate for more control, or alternatively, return {@link Callable}.
 	 * @param predicate the predicate to use
-	 * @since 1.3
+	 * @since 1.3.0
 	 */
 	public void setBlockingMethodPredicate(@Nullable Predicate<HandlerMethod> predicate) {
 		this.blockingMethodPredicate = ((predicate != null) ? predicate : (handlerMethod) -> false);

spring-graphql/src/main/java/org/springframework/graphql/server/TimeoutWebGraphQlInterceptor.java

@@ -31,7 +31,7 @@
  * until the response stream is established. Transport-specific timeouts are
  * configurable on the transport handlers directly.
  * @author Brian Clozel
- * @since 1.4
+ * @since 1.4.0
  */
 public class TimeoutWebGraphQlInterceptor implements WebGraphQlInterceptor {
 

spring-graphql/src/main/java/org/springframework/graphql/server/webflux/GraphQlWebSocketHandler.java

@@ -105,7 +105,7 @@ public GraphQlWebSocketHandler(
 	 * the WebSocket for the {@code "connection_ini"} message from the client.
 	 * @param keepAliveDuration how frequently to send ping messages when no
 	 * other messages are sent
-	 * @since 1.3
+	 * @since 1.3.0
 	 */
 	public GraphQlWebSocketHandler(
 			WebGraphQlHandler graphQlHandler, CodecConfigurer codecConfigurer,

spring-graphql/src/main/java/org/springframework/graphql/server/webmvc/GraphQlWebSocketHandler.java

@@ -129,7 +129,7 @@ public GraphQlWebSocketHandler(
 	 * the WebSocket for the {@code "connection_ini"} message from the client.
 	 * @param keepAliveDuration how frequently to send ping messages when no
 	 * other messages are sent
-	 * @since 1.3
+	 * @since 1.3.0
 	 */
 	public GraphQlWebSocketHandler(
 			WebGraphQlHandler graphQlHandler, HttpMessageConverter<Object> converter,

src/checkstyle/checkstyle.xml

@@ -20,5 +20,13 @@
 			<property name="accessModifiers" value="public,protected"/>
 			<property name="allowMissingReturnTag" value="true"/>
 		</module>
+        <module name="com.puppycrawl.tools.checkstyle.checks.regexp.RegexpSinglelineJavaCheck">
+            <property name="id" value="atSinceVersionConvention"/>
+            <property name="format" value="\@since \d+\.\d+$"/>
+            <property name="maximum" value="0"/>
+            <property name="message"
+                      value="@since should use Major.Minor.Patch version convention"/>
+            <property name="ignoreComments" value="false"/>
+        </module>
 	</module>
 </module>