Skip to content

Add Unwrap methods for bolt errors#632

Open
arxeiss wants to merge 1 commit into
neo4j:6.xfrom
arxeiss:add-unwrap-methods
Open

Add Unwrap methods for bolt errors#632
arxeiss wants to merge 1 commit into
neo4j:6.xfrom
arxeiss:add-unwrap-methods

Conversation

@arxeiss

@arxeiss arxeiss commented Feb 27, 2025

Copy link
Copy Markdown

Error types are in internal package, so casting is not possible. And because of missing Unwrap method, the errors.Is() doesn't work either.

Closing #631

@arxeiss

arxeiss commented May 22, 2025

Copy link
Copy Markdown
Author

Is there anything I can do to make this PR to be merged?

@StephenCathcart
StephenCathcart changed the base branch from 5.0 to 6.x May 27, 2025 08:50
@StephenCathcart

Copy link
Copy Markdown
Contributor

Hi @arxeiss,

Thank you for taking the time to contribute. We appreciate all community engagement.
Before we review a change, we require that users have signed our Contributor License Agreement(CLA). This allows the "whitelist-check (pull-request)" part of our build process to pass.
For more details on signing our CLA, please see: https://neo4j.com/developer/cla/. Specifically, the 'How to Sign' section.

I've also updated this PR to point towards our 6.x branch, as 5.28 is now Long-Term Support (LTS).

@StephenCathcart

Copy link
Copy Markdown
Contributor

Hey @arxeiss, thanks for signing our CLA. The change looks very reasonable. We're currently in the process of updating our pipelines for our upcoming 6.0 release, so I'll be sure to include this PR with that. Thanks again for the contribution 👍.

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