-
Notifications
You must be signed in to change notification settings - Fork 3.5k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Go][CI] Remove Go related test CI except integration test #43873
Comments
Closed
raulcd
added a commit
to raulcd/arrow
that referenced
this issue
Sep 17, 2024
raulcd
added a commit
to raulcd/arrow
that referenced
this issue
Sep 18, 2024
kou
pushed a commit
that referenced
this issue
Sep 19, 2024
### Rationale for this change The Go implementation is moving to apache/arrow-go from go/ in apache/arrow. ### What changes are included in this PR? CI has been migrated to apache/arrow-go repository. ### Are these changes tested? No ### Are there any user-facing changes? No * GitHub Issue: #43873 Authored-by: Raúl Cumplido <raulcumplido@gmail.com> Signed-off-by: Sutou Kouhei <kou@clear-code.com>
Issue resolved by pull request 44143 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Describe the enhancement requested
After apache/arrow-go#1.
We can remove the followings:
We should not remove integration test. It's still needed after apache/arrow-go#1 to ensure compatibility. (We need to update integration test but it's out-of-scope of this.)
Component(s)
Continuous Integration, Go
The text was updated successfully, but these errors were encountered: