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

Test patched async-io on more platforms #79

Merged
merged 1 commit into from
Apr 10, 2023
Merged

Test patched async-io on more platforms #79

merged 1 commit into from
Apr 10, 2023

Conversation

taiki-e
Copy link
Collaborator

@taiki-e taiki-e commented Jan 22, 2023

Split from #78

Blocked on smol-rs/async-io#110
Also the increase in time taken to test with vmactions is annoying.

after

@taiki-e taiki-e marked this pull request as draft January 22, 2023 15:07
@taiki-e taiki-e force-pushed the taiki-e/ci2 branch 2 times, most recently from b98bc7c to cf77093 Compare February 6, 2023 13:03
@taiki-e taiki-e force-pushed the taiki-e/ci2 branch 2 times, most recently from a6bc97a to 2a2846d Compare March 26, 2023 09:55
@taiki-e taiki-e marked this pull request as ready for review March 26, 2023 10:09
Copy link
Member

@notgull notgull left a comment

Choose a reason for hiding this comment

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

Thanks!

Re: the vmactions taking forever. Maybe can only run them in specific circumstances? (Such as only if any of the kqueue-related files were affected).

@taiki-e taiki-e merged commit 1e966a0 into master Apr 10, 2023
@taiki-e taiki-e deleted the taiki-e/ci2 branch April 10, 2023 06:34
@taiki-e
Copy link
Collaborator Author

taiki-e commented Apr 10, 2023

As for vmactions, I was thinking of trying another action that may be faster.

Such as only if any of the kqueue-related files were affected

We have tried something similar in tokio (for loom-related files). However, the effect of this may be smaller, because these targets may also be affected by changes in tests and doc examples.

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

Successfully merging this pull request may close these issues.

None yet

2 participants