Skip to content
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

GH-11: Add test CI: macOS #86

Merged
merged 1 commit into from
Sep 7, 2024
Merged

GH-11: Add test CI: macOS #86

merged 1 commit into from
Sep 7, 2024

Conversation

kou
Copy link
Member

@kou kou commented Sep 5, 2024

Fix GH-11

This part:
https://github.com/apache/arrow/blob/58415d1fac50cb829b3dcf08526033d6db8c30db/.github/workflows/go.yml#L305-L360

This also imports ci/scripts/go_{build,test}.sh from apache/arrow.

This also introduce ShellCheck and shfmt with pre-commit.

This also adds apache/arrow-testing and apache/parquet-testing submodules.

@kou kou force-pushed the ci-macos branch 6 times, most recently from c159498 to f4f2ddc Compare September 5, 2024 08:38
@kou
Copy link
Member Author

kou commented Sep 5, 2024

@kou kou changed the title Add test CI: macOS GH-11: Add test CI: macOS Sep 5, 2024
@zeroshade
Copy link
Member

zeroshade commented Sep 5, 2024

Before we can truly claim that the CI is working, we need to update the package path of the go.mod file and all of the imports in the files to point to itself here instead of the old paths.

We can probably leverage the same sed scripts we use for updating the major version in the imports to quickly update all of the files to use the new import path

@kou
Copy link
Member Author

kou commented Sep 5, 2024

OK. How about #88?

Fix apacheGH-11

This part:
https://github.com/apache/arrow/blob/58415d1fac50cb829b3dcf08526033d6db8c30db/.github/workflows/go.yml#L305-L360

This also imports `ci/scripts/go_{build,test}.sh` from apache/arrow.

This also introduce ShellCheck and shfmt with pre-commit.

This also adds apache/arrow-testing and apache/parquet-testing
submodules.
@kou
Copy link
Member Author

kou commented Sep 7, 2024

@kou kou merged commit d486377 into apache:main Sep 7, 2024
@kou kou deleted the ci-macos branch September 7, 2024 05:51
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.

Add test CI: macOS
2 participants