search

apache / arrow

Apache Arrow is a multi-language toolbox for accelerated data interchange and in-memory processing
https://arrow.apache.org/
Apache License 2.0
13.89k stars 3.38k forks source link

[JS][Release] Stop re-testing JS packages when uploading to npm #41650

Open raulcd opened 1 month ago

raulcd commented 1 month ago

Describe the enhancement requested

As part of the release process we test all the packages on several CI jobs for different OS and environments.

We also do a lot of integration tests and release verification.

As part of the release verification we do test JS packages.

Once the release has been approved we are executing the npm-release.sh where we re-test again JS.

JS is the only package were once the release is approved we test again. Is this really necessary?

The post-release task for JS could be reduced if we avoid testing which feels unnecessary at this point.

Component(s)

JavaScript, Release

raulcd commented 1 month ago

@kou @domoritz does this feel reasonable to you?

kou commented 1 month ago

It makes sense. Can we also upload yarn build results as release artifacts like wheels so that we can just publish them in our post release script?

domoritz commented 1 month ago

Makes sense to me not to test again.