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

[Auth restructure 4] Fix examples/tests/docs and CI #216

Merged
merged 10 commits into from
Jun 19, 2021

Conversation

marioortizmanero
Copy link
Collaborator

@marioortizmanero marioortizmanero commented Jun 19, 2021

Part 4 of #207's split up. This fixes many other things related to the rewrite: the examples, tests, documentation, and CI.

The main library should pass the tests & CI, and the examples should work after this PR.

@marioortizmanero marioortizmanero changed the base branch from master to auth-rewrite-part3 June 19, 2021 10:23
@marioortizmanero
Copy link
Collaborator Author

I'm currently updating the changelog to include a small guide on how to upgrade to the new version. I've opened the #218 issue for those with questions regarding the update, and I'll mention said issue in the changelog as well.

@ramsayleung
Copy link
Owner

Is there any required order for this [Auth restructure part N] PR to merge? Does one PR reply to another? If not, is it fine to merge individually when one PR review approves and CI pass?

@marioortizmanero
Copy link
Collaborator Author

You can merge as you like, GitHub should handle it correctly. Though the safest way to go is first 4, then 3, then 2, then 1, then separate-crates.

CHANGELOG.md Show resolved Hide resolved
CHANGELOG.md Show resolved Hide resolved
@ramsayleung ramsayleung merged commit e1c6c44 into auth-rewrite-part3 Jun 19, 2021
@ramsayleung ramsayleung deleted the auth-rewrite-part4 branch June 19, 2021 14:21
@ramsayleung
Copy link
Owner

Looks good to me, merged :)

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