Skip to content

ci: run rust-tests on windows#92

Merged
xushiyan merged 2 commits intoapache:mainfrom
crrow:feat/run-ci-on-windows
Mar 21, 2026
Merged

ci: run rust-tests on windows#92
xushiyan merged 2 commits intoapache:mainfrom
crrow:feat/run-ci-on-windows

Conversation

@crrow
Copy link
Copy Markdown
Contributor

@crrow crrow commented Jul 24, 2024

Description

Setup ci rust tests on windows

closes #90

How are the changes test-covered

  • N/A
  • Automated tests (unit and/or integration tests)
  • Manual tests
    • Details are described below

@crrow crrow force-pushed the feat/run-ci-on-windows branch 2 times, most recently from 340f06b to 13d4ca9 Compare July 24, 2024 15:56
@crrow crrow changed the title Feat/run ci on windows chore: run ci on windows Jul 24, 2024
@github-actions github-actions Bot added the ci label Jul 24, 2024
@xushiyan xushiyan added this to the release-0.2.0 milestone Jul 25, 2024
@xushiyan xushiyan changed the title chore: run ci on windows ci: run rust-tests on windows Jul 25, 2024
Copy link
Copy Markdown
Member

@xushiyan xushiyan left a comment

Choose a reason for hiding this comment

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

Screenshot 2024-07-24 at 9 35 50 PM

@crrow thanks for the fix! interestingly the CI job wasn't even triggered. any idea what's going on?

Comment thread .github/workflows/ci.yml Outdated
@xushiyan
Copy link
Copy Markdown
Member

@crrow i suspect your previous approach using binstall could work but the github action was not whitelisted by asf so the whole workflow is not triggered. if you can verify that binstall approach in your own fork and get CI to pass, then i can reach out to have that white listed.

@crrow
Copy link
Copy Markdown
Contributor Author

crrow commented Jul 27, 2024

@crrow i suspect your previous approach using binstall could work but the github action was not whitelisted by asf so the whole workflow is not triggered. if you can verify that binstall approach in your own fork and get CI to pass, then i can reach out to have that white listed.

ok, i'll check it out later.

@crrow crrow force-pushed the feat/run-ci-on-windows branch from 6300fc3 to 0b991e3 Compare July 27, 2024 09:31
@crrow
Copy link
Copy Markdown
Contributor Author

crrow commented Jul 27, 2024

@crrow i suspect your previous approach using binstall could work but the github action was not whitelisted by asf so the whole workflow is not triggered. if you can verify that binstall approach in your own fork and get CI to pass, then i can reach out to have that white listed.

ok, i'll check it out later.

Hello @xushiyan , I setup the workflow, have a check when you are free.

https://github.com/crrow/hudi-rs/actions/runs/10122608881/job/27994952767

Comment thread .github/workflows/ci.yml Outdated
@xushiyan xushiyan modified the milestones: release-0.2.0, release-0.3.0 Sep 15, 2024
@xushiyan xushiyan added the help wanted Extra attention is needed label Dec 13, 2024
@xushiyan xushiyan modified the milestones: release-0.3.0, release-0.4.0 Jan 23, 2025
@xushiyan xushiyan modified the milestones: release-0.4.0, release-0.5.0 Mar 21, 2025
@xushiyan xushiyan force-pushed the feat/run-ci-on-windows branch from 0b991e3 to b5d2311 Compare October 3, 2025 21:57
@codecov
Copy link
Copy Markdown

codecov Bot commented Oct 3, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 84.47%. Comparing base (f2441da) to head (fbc1859).
⚠️ Report is 2 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #92   +/-   ##
=======================================
  Coverage   84.47%   84.47%           
=======================================
  Files          74       74           
  Lines        4778     4778           
=======================================
  Hits         4036     4036           
  Misses        742      742           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@xushiyan xushiyan force-pushed the feat/run-ci-on-windows branch from e10aef1 to 61a33ce Compare October 3, 2025 23:07
@xushiyan xushiyan force-pushed the feat/run-ci-on-windows branch from 61a33ce to c5dfa75 Compare March 21, 2026 17:42
Temporarily disable Windows tests until issue apache#445 is resolved.
@xushiyan xushiyan merged commit 9f6505b into apache:main Mar 21, 2026
15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci help wanted Extra attention is needed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Support rust-tests for windows in CI

2 participants