Skip to content
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

Add GitHub CI and fix all found issues #163

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

tisonkun
Copy link

@seanmonstar

First I'd like to add a GitHub CI coverage for this repo, and then I go through all the files for fmt and clippy.

Let me know if it's desired or if anything I can help to move this crate to 1.0.

Signed-off-by: tison <wander4096@gmail.com>
Signed-off-by: tison <wander4096@gmail.com>
@seanmonstar
Copy link
Member

Thanks for offering to help! To see what's currently blocking this library, take a look at the pinned #153.

@tisonkun
Copy link
Author

@seanmonstar Thanks for pointing out. I'm spending some time on the issue now.

What do you think of this PR? Generally, I'd start with a CI-covered workspace to help make progress more confidently.

# 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.

2 participants