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.

Sign up for GitHub

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

Jump to bottom

Since using nextest we are no longer running the doctests #1509

Closed
tillrohrmann opened this issue May 13, 2024 · 0 comments · Fixed by #1771
Closed

Since using nextest we are no longer running the doctests #1509

tillrohrmann opened this issue May 13, 2024 · 0 comments · Fixed by #1771
Assignees
@pcholakov
Labels
good first issue Good for newcomers onboarding

Comments

@tillrohrmann
Copy link
Contributor

Nextest apparently does not support running doctests nextest-rs/nextest#16. Therefore, some of our examples in the docs are broken. It would be great to get this functionality back.
@pcholakov pcholakov self-assigned this Jul 30, 2024
pcholakov added a commit to pcholakov/restate that referenced this issue Jul 30, 2024
@pcholakov
Run doctests using cargo test
45a9608 
Fixes restatedev#1509
pcholakov added a commit to pcholakov/restate that referenced this issue Jul 30, 2024
@pcholakov
Run doctests using cargo test
5a9e806 
Fixes restatedev#1509
pcholakov added a commit to pcholakov/restate that referenced this issue Jul 30, 2024
@pcholakov
Run doctests using cargo test
b4e04df 
Fixes restatedev#1509
@pcholakov pcholakov mentioned this issue Jul 30, 2024
Merged
pcholakov added a commit that referenced this issue Jul 31, 2024
@pcholakov
Run doctests using cargo test (#1771)
5f36dcb 
Fixes #1509
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@pcholakov pcholakov

Labels
good first issue Good for newcomers onboarding
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Run doctests using cargo test pcholakov/restate
2 participants
@pcholakov @tillrohrmann