Document Java.Net HTTP status values - #628
Conversation
Refs #232 Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
There was a problem hiding this comment.
Pull request overview
This PR improves the completeness of the Java.Net XML API documentation by replacing placeholder text in Java.Net.HttpStatus with concrete summaries for each mapped HTTP status value (mirroring Android java.net.HttpURLConnection constants), supporting the broader effort in #232 to eliminate “To be added” placeholders.
Changes:
- Replaced all remaining
To be added.placeholders inJava.Net.HttpStatusfield docs with specific HTTP status summaries. - Ensured coverage across all 36
Java.Net.HttpStatusvalues mapped fromjava.net.HttpURLConnection.
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
|
@dalexsoto review |
dalexsoto
left a comment
There was a problem hiding this comment.
ServerError maps to Android's deprecated HTTP_SERVER_ERROR, which is misplaced and replaced by InternalError, but the new summary presents it as an ordinary 500 value. Put the deprecation warning and replacement in the published summary, or mark the enum value obsolete upstream and regenerate.
Explain that ServerError maps to the misplaced deprecated Android constant and direct users to InternalError. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> Copilot-Session: 3acbf5b4-9edd-468a-9051-d61e7f774c07
|
Addressed in 4a065d3: ServerError now explicitly warns that the mapped Android constant is deprecated and misplaced, and directs users to InternalError. XML parsing, CRLF-only line endings, and git diff --check all pass. |
|
Addressed in 4a065d3. The published |
dalexsoto
left a comment
There was a problem hiding this comment.
ServerError now clearly documents the deprecated mapping and valid InternalError replacement; the status values and generated output are correct.
Summary
Java.Net.HttpStatusvalues mapped fromjava.net.HttpURLConnectionconstants.Refs #232
Sources
java.net.HttpURLConnectionconstantsValidation
docs/xml/Java.Net/HttpStatus.xmlwith PowerShell XML parsing.git diff --check.