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

feat: add ci config, rustfmt, dependabot, etc. #2

Merged
merged 1 commit into from
Mar 21, 2024
Merged

Conversation

bitwalker
Copy link
Contributor

No description provided.

@bitwalker bitwalker requested a review from bobbinth March 21, 2024 16:41
@bitwalker bitwalker self-assigned this Mar 21, 2024
@bitwalker
Copy link
Contributor Author

@bobbinth I'll merge this now, so that any subsequent PRs can pick up the CI checks and what not, but I believe this addresses all of the changes you wanted in #1. So aside from a cursory review, things should be ready to publish to crates.io whenever you are ready

@bitwalker bitwalker merged commit e2c3b54 into main Mar 21, 2024
12 checks passed
@bitwalker bitwalker deleted the bitwalker/ci branch March 21, 2024 18:08
@bobbinth
Copy link
Contributor

I fixed some minor issues and published to crates.io. Docs show some error, though also seem to have imported the readme correctly.

@bitwalker
Copy link
Contributor Author

I fixed some minor issues and published to crates.io. Docs show some error, though also seem to have imported the readme correctly.

I'll take a look shortly and open a PR with a fix, not sure what went wrong, must be due to the changes I made though, since the docs built fine yesterday.

@bobbinth
Copy link
Contributor

No rush on this at all (we can fix it later down the road). One thing: I did alter the path of the include_str macro - otherwise cargo publish didn't work (you can check it with the --dry-run flag).

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