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

fix(ci): add doc tests to CI #383

Merged
merged 4 commits into from
Nov 7, 2024
Merged

fix(ci): add doc tests to CI #383

merged 4 commits into from
Nov 7, 2024

Conversation

Serial-ATA
Copy link
Contributor

nextest doesn't support doc tests currently. We need to make sure that examples in the docs compile.

Needs #382 since it fixes another failing doc test

@Serial-ATA
Copy link
Contributor Author

This'll keep failing while the job macro is being iterated on. I'll just make this a draft until it stops breaking.

@Serial-ATA Serial-ATA marked this pull request as draft October 24, 2024 18:56
@drewstone
Copy link
Contributor

@Serial-ATA I fixed up the docker docs tests here. The context doc test doesn't work yet.

Serial-ATA and others added 4 commits November 7, 2024 10:22
`nextest` doesn't support doc tests currently. We need to make sure that examples in the docs compile.
@Serial-ATA Serial-ATA changed the title fix: add doc tests to CI fix(ci): add doc tests to CI Nov 7, 2024
@Serial-ATA Serial-ATA marked this pull request as ready for review November 7, 2024 15:22
@drewstone drewstone merged commit 88965db into main Nov 7, 2024
13 checks passed
@webb-spider webb-spider bot mentioned this pull request Nov 7, 2024
@Serial-ATA Serial-ATA deleted the serial/doc-tests branch February 14, 2025 16:41
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
Status: Completed ✅
Development

Successfully merging this pull request may close these issues.

2 participants