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

add GitHub action for push and pr #39

Merged
merged 1 commit into from
May 2, 2020
Merged

add GitHub action for push and pr #39

merged 1 commit into from
May 2, 2020

Conversation

rizsotto
Copy link

Adding continuous integration for this project. (Currently strict checking is disabled.)

@rizsotto
Copy link
Author

Hey @inejge , is there a chance for you to look to this PR?

@inejge
Copy link
Owner

inejge commented May 1, 2020

Hi, thanks for the PR and sorry for the delay. The crate has been ported to async/await and modernized code-wise; also formatted and passed through Clippy. So, it should a) compile without warnings, b) pass cargo fmt tests, c) pass Clippy tests. If you're still interested, could you update the PR to turn on all of those, and also limit Clippy to stable?

@rizsotto
Copy link
Author

rizsotto commented May 2, 2020

I've updated this PR as you suggested, but Clippy is still failing.

@inejge
Copy link
Owner

inejge commented May 2, 2020

It's probably this issue, so I'm simply going to remove Clippy checking -- I do use Clippy, but can easily live with running it manually from time to time.

@inejge inejge merged commit 573cc5b into inejge:master May 2, 2020
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.

2 participants