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

Fix compilation with use_tokio feature #61

Merged
merged 2 commits into from
Mar 11, 2020

Conversation

pheki
Copy link
Contributor

@pheki pheki commented Mar 11, 2020

Apparently I've introduced a regression in #58 (sorry!) because I didn't run the tests with features (and travis didn't run as seen in #59 (comment)).

This fixes that regression by compiling get_value_from_file also when feature = "use_tokio".

I also took the opportunity to use ? instead of all remaining try!() macros.

@rust-highfive
Copy link

r? @posborne

(rust_highfive has picked a reviewer for you, use r? to override)

@nastevens
Copy link
Member

nastevens commented Mar 11, 2020

It looks like the abuse flagging by Travis CI is a known issue. From there it sounds like you can email support@travis-ci.com to get your account flag updated. It'd be really nice to have CI running on your PRs! Either way, the updates are much appreciated!

@nastevens nastevens merged commit e61d4e1 into rust-embedded:master Mar 11, 2020
@pheki
Copy link
Contributor Author

pheki commented Mar 12, 2020

Sent the email! Waiting for their fix now ^^

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

Successfully merging this pull request may close these issues.

4 participants