[GitHub] [incubator-pekko-http] pjfanning commented on a diff in pull request #87: Update naming for status codes 413 and 422

["pjfanning (via GitHub)" <gi...@apache.org>]

pjfanning commented on code in PR #87:
URL: https://github.com/apache/incubator-pekko-http/pull/87#discussion_r1118119683


##########
http-core/src/main/java/org/apache/pekko/http/javadsl/model/StatusCodes.java:
##########
@@ -66,10 +66,16 @@ private StatusCodes() {}
     public static final StatusCode GONE = org.apache.pekko.http.scaladsl.model.StatusCodes.Gone();
     public static final StatusCode LENGTH_REQUIRED = org.apache.pekko.http.scaladsl.model.StatusCodes.LengthRequired();
     public static final StatusCode PRECONDITION_FAILED = org.apache.pekko.http.scaladsl.model.StatusCodes.PreconditionFailed();
+    public static final StatusCode CONTENT_TOO_LARGE = org.apache.pekko.http.scaladsl.model.StatusCodes.ContentTooLarge();
+
+    /**
+     * @deprecated deprecated in favor of CONTENT_TOO_LARGE

Review Comment:
   Could you include `since = "1.0.0"` on all new deprecations?



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@pekko.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@pekko.apache.org
For additional commands, e-mail: notifications-help@pekko.apache.org