Skip to content

GH-46587: [CI][JS] Remove JS related test CI#46588

Merged
kou merged 2 commits into
apache:mainfrom
raulcd:GH-46587
May 26, 2025
Merged

GH-46587: [CI][JS] Remove JS related test CI#46588
kou merged 2 commits into
apache:mainfrom
raulcd:GH-46587

Conversation

@raulcd
Copy link
Copy Markdown
Member

@raulcd raulcd commented May 26, 2025

Rationale for this change

As part of moving the JavaScript implementation to https://github.com/apache/arrow-js we have to remove CI related workflows and scripts from the main repository.

What changes are included in this PR?

Remove JS related CI and build scripts.

Are these changes tested?

No

Are there any user-facing changes?

No

@github-actions
Copy link
Copy Markdown

⚠️ GitHub issue #46587 has been automatically assigned in GitHub to PR creator.

@github-actions github-actions Bot added the awaiting committer review Awaiting committer review label May 26, 2025
Copy link
Copy Markdown
Member

@kou kou left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1

@kou kou merged commit f88c19f into apache:main May 26, 2025
32 of 34 checks passed
@kou kou removed the awaiting committer review Awaiting committer review label May 26, 2025
@github-actions github-actions Bot added the awaiting merge Awaiting merge label May 26, 2025
@conbench-apache-arrow
Copy link
Copy Markdown

After merging your PR, Conbench analyzed the 4 benchmarking runs that have been run so far on merge-commit f88c19f.

There were no benchmark performance regressions. 🎉

The full Conbench report has more details. It also includes information about 1 possible false positive for unstable benchmarks that are known to sometimes produce them.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

awaiting merge Awaiting merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants