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

ci: use taiki-e/install-action to install tools from binary #852

Merged
merged 5 commits into from
Jan 4, 2025

Conversation

xxchan
Copy link
Member

@xxchan xxchan commented Dec 27, 2024

instead of compiling from source

need to wait for INFRA approval https://issues.apache.org/jira/browse/INFRA-26400

instead of compiling from source

need to wait for INFRA approval https://issues.apache.org/jira/browse/INFRA-26400

Signed-off-by: xxchan <xxchan22f@gmail.com>
Comment on lines +53 to +54
- name: Check toml format
run: make check-toml
Copy link
Member Author

Choose a reason for hiding this comment

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

Previously we only installed taplo, but didn't run it

@xxchan xxchan marked this pull request as draft December 27, 2024 07:54
@xxchan xxchan marked this pull request as ready for review January 3, 2025 09:41
Signed-off-by: xxchan <xxchan22f@gmail.com>
@xxchan
Copy link
Member Author

xxchan commented Jan 3, 2025

INFRA has approved the action

run: make install-taplo-cli
uses: taiki-e/install-action@v2
with:
tool: taplo-cli@0.9.3
Copy link
Member Author

Choose a reason for hiding this comment

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

Bump version to fix this issue
image

Signed-off-by: xxchan <xxchan22f@gmail.com>
Comment on lines -65 to +70
run: make cargo-sort
run: cargo sort -c -w
Copy link
Member Author

Choose a reason for hiding this comment

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

meet an installation error
image

Signed-off-by: xxchan <xxchan22f@gmail.com>
Copy link
Member

@Xuanwo Xuanwo left a comment

Choose a reason for hiding this comment

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

Thank you @xxchan for working on this!

@Xuanwo Xuanwo merged commit e5bdbfc into apache:main Jan 4, 2025
16 checks passed
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.

None yet

2 participants