Skip to content

Document JDBC driver properties - #551

Open
jonathanpeppers wants to merge 1 commit into
mainfrom
jonathanpeppers-document-java-sql-apis
Open

Document JDBC driver properties#551
jonathanpeppers wants to merge 1 commit into
mainfrom
jonathanpeppers-document-java-sql-apis

Conversation

@jonathanpeppers

Copy link
Copy Markdown
Member

Refs #189

Documents the return values for the five public DriverPropertyInfo fields from the Android reference.

Sources

Validation

  • Parsed all 55 docs/xml/Java.Sql/*.xml files with PowerShell XML parsing.
  • Confirmed zero lone LF bytes in docs/xml/Java.Sql (CRLF preserved).
  • Ran git diff --check.
  • Confirmed 200 To be added. placeholders remain in docs/xml/Java.Sql.

Refs #189

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This pull request updates the XML documentation for Java.Sql.DriverPropertyInfo to replace “To be added.” placeholders with concrete <value> descriptions derived from the Android reference, improving the usefulness of the generated API docs for JDBC driver property metadata.

Changes:

  • Documented the return/value semantics for Choices, Description, Name, Required, and Value in DriverPropertyInfo.
  • Clarified null cases for nullable properties (e.g., unrestricted choices, unknown value).

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@jonathanpeppers

Copy link
Copy Markdown
Member Author

@dalexsoto review

1 similar comment
@jonathanpeppers

Copy link
Copy Markdown
Member Author

@dalexsoto review

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants