Skip to content

fix(fs): use smol::block_on for drop handling of File #768

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Merged
merged 2 commits into from
May 9, 2020

Conversation

dignifiedquire
Copy link
Member

This avoids nesting of task::block_on calls.

Ref #766

@dignifiedquire dignifiedquire requested a review from yoshuawuyts May 9, 2020 09:44
@0x8f701
Copy link

0x8f701 commented May 9, 2020

I'd like to suggest having: 1.6.0-beta.2 so that more people can test

0x8f701 pushed a commit to 0x8f701/casbin-rs that referenced this pull request May 9, 2020
Copy link
Contributor

@yoshuawuyts yoshuawuyts left a comment

Choose a reason for hiding this comment

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

LGTM -- though I'm wondering whether it'd make sense to use smol::block_on as the inner impl of our task::block_on?

@dignifiedquire dignifiedquire merged commit e20b0f0 into master May 9, 2020
@dignifiedquire dignifiedquire deleted the fix/file-block branch May 9, 2020 14:28
0x8f701 added a commit to casbin/casbin-rs that referenced this pull request May 10, 2020
* upgrade async-std to 1.6 beta

* add ip feature

* add glob feature

* add cacehd feature

* fix runtime-tokio clippy warnings

* add watcher feature

* don't clone if watcher feature has been disabled

* use async-rs/async-std#768 for fixing tests

* activate all features for bench test over previous version

* benchmark pull_request and decrease the threshold

* switch to github actions

* fix typo

* Fix: cargo test features doesn't like space

* better management of feature:logging and feature:watcher

* use async-std/master for testing

* add basic wasm32 support

* use runtime-async-std for wasm32 test

* fix clippy warnings

* fix typo

* make other tests avalaible only on no-wasm32 target

* finish split code into features && bump version
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants