Skip to content

Conversation

@AdamGS
Copy link
Contributor

@AdamGS AdamGS commented Sep 15, 2025

Which issue does this PR close?

Rationale for this change

Should make CI faster. For example currently just installing cargo-msrv takes >5 minutes, with this PR its ~4 seconds.

What changes are included in this PR?

Are these changes tested?

Are there any user-facing changes?

@github-actions github-actions bot added the development-process Related to development process of DataFusion label Sep 15, 2025
@AdamGS AdamGS force-pushed the adamg/use-taiki-e-install-action-ci branch from 9a92103 to 5a63a6c Compare September 15, 2025 12:27
@AdamGS AdamGS marked this pull request as ready for review September 15, 2025 12:29
- uses: actions/checkout@08c6903cd8c0fde910a37f88322edcfb5dd907a8 # v5.0.0
- name: Install cargo-audit
run: cargo install cargo-audit
uses: taiki-e/install-action@v2
Copy link
Collaborator

Choose a reason for hiding this comment

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

should we put the version hash?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

sure, give me a second

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done

@AdamGS AdamGS requested a review from blaginin September 15, 2025 12:51
Copy link
Collaborator

@blaginin blaginin left a comment

Choose a reason for hiding this comment

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

thanks!

@blaginin
Copy link
Collaborator

blaginin commented Sep 15, 2025

I think in the future we could also use nextest to run tests!

One thing before merging, @Omega359, you wrote:

taiki-e/install-action@nextest is not an allowed action

I've read asf guide and I don't see it's being not allowed - is there something I'm missing?

@Omega359
Copy link
Contributor

It failed when I had it enabled. I somewhat doubt it would run well anyways given the machine constraints

@blaginin
Copy link
Collaborator

blaginin commented Sep 15, 2025

Got it! I think it's working now (and actually uses less resources as tools are not built), so going to merge if that's ok with everyone. I think we can iterate on top (as in adding nextest and some more tools)?

@blaginin blaginin merged commit 7d08c27 into apache:main Sep 15, 2025
28 checks passed
@AdamGS AdamGS deleted the adamg/use-taiki-e-install-action-ci branch September 15, 2025 20:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

development-process Related to development process of DataFusion

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Use taiki-e/install-action in CI

3 participants