kou commented on PR #320: URL: https://github.com/apache/arrow-js/pull/320#issuecomment-3483579134
We already have integration test CI job: https://github.com/apache/arrow-js/blob/32bb48b4d80ec4fdb632f175826c687710d98545/.github/workflows/test.yaml#L143-L210 We can change Archery something like the following: ```diff diff --git a/.github/workflows/test.yaml b/.github/workflows/test.yaml index 1370581..539e2c0 100644 --- a/.github/workflows/test.yaml +++ b/.github/workflows/test.yaml @@ -193,6 +193,9 @@ jobs: uses: actions/setup-python@e797f83bcb11b83ae66e0230d6156d7c80228e7c # v6.0.0 with: python-version: 3 + - name: Patch Archery + run: | + patch -p1 -i js/.../archery.patch - name: Setup Archery run: pip install -e dev/archery[docker] - name: Execute Docker Build ``` If it works well, we can open a PR to https://github.com/apache/arrow after we merge this so that we don't need to apply a patch locally. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
