Skip to content

Conversation

@sivchari
Copy link
Contributor

@sivchari sivchari commented Mar 19, 2025

WHAT

Added replace directive.

WHY

Currently, wrench got the build failure since the xiter uses invalid syntax. (ref: spheric-cloud/xiter#1)
The pr referred above will correct it, but we don't know when it's merged.
So I added replace directive to avoid this error. Since, we've already had the flow to publish binary, it's not problem to have replace directive in itself.

Signed-off-by: sivchari <shibuuuu5@gmail.com>
@vvakame
Copy link
Collaborator

vvakame commented Apr 28, 2025

@sivchari thanks to your contributions!
I can't reproduce this issue. could you provide some details?

@apstndb
Copy link
Contributor

apstndb commented Aug 27, 2025

Since there is no comment from the author, I will comment instead. My understanding is that this was a temporary bug that existed during the development of Go 1.25, and that discussions have confirmed it does not exist in any officially released version of Go.
If there are no objections, I think it's fine to close this.

apstndb/gsqlutils#1 (comment)
spheric-cloud/xiter#1 (comment)

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.

3 participants