Skip to content

Clear, scoped (docs), and says what changed. Matches common Conventional Commits style.#302

Merged
transistive merged 2 commits into
mainfrom
fix/readme-https-documentation
Apr 13, 2026
Merged

Clear, scoped (docs), and says what changed. Matches common Conventional Commits style.#302
transistive merged 2 commits into
mainfrom
fix/readme-https-documentation

Conversation

@p123-stack
Copy link
Copy Markdown
Collaborator

States why (removed in 3.3.0), what in the code/docs (ClientBuilder schemes), and what you updated (Step 2, feature list, matrix, etc.).
Fixes #300: Correct for GitHub—when the PR merges into the default branch, it should auto-close issue #300. Putting it on its own last line is fine.

HTTP/HTTPS drivers were dropped in v3.3.0; ClientBuilder only
supports Bolt and Neo4j URI schemes. Update Step 2, feature list,
scheme matrix, and related sections accordingly.

Fixes #300
Comment thread README.md Outdated
#### Running multiple queries at once

The `runStatements` method will run all the statements at once. This method is an essential tool to reduce the number of database calls, especially when using the HTTP protocol.
The `runStatements` method will run all the statements at once. This method is an essential tool to reduce the number of database round trips when batching work.
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

In bolt we cannot send multiple queries at once. We can just remove this section

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Thanks — agreed. I’ve removed that whole subsection. The README no longer suggests batching multiple statements in one round trip for Bolt.

Comment thread README.md Outdated
(*) Needed to implement the bolt protocol

(**) Needed to implement the http protocol
(**) Required by the package for JSON handling
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

I assume this can just be removed

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

I’ve removed those footnotes and the * / ** / *** markers on the extension lines so the requirements list stays simple.

@transistive transistive marked this pull request as ready for review April 13, 2026 07:21
@transistive transistive merged commit c814ca0 into main Apr 13, 2026
15 of 16 checks passed
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.

Documentation still references https connection

2 participants