Skip to content
This repository has been archived by the owner on Nov 2, 2023. It is now read-only.

Add mdbook tests to CI #2

Open
alice-i-cecile opened this issue Sep 26, 2021 · 3 comments
Open

Add mdbook tests to CI #2

alice-i-cecile opened this issue Sep 26, 2021 · 3 comments
Labels
C: ci Continuous integration D: high Quite complex or very technically challenging I: low Nice to have S: blocked-upstream No progress can be made on this until an upstream dependency fixes something T: enhancement A new feature

Comments

@alice-i-cecile
Copy link
Contributor

See: https://rust-lang.github.io/mdBook/cli/test.html

@alice-i-cecile
Copy link
Contributor Author

Can probably just add to CI's main.rs

@alice-i-cecile alice-i-cecile added D: moderate Some thought and experience needed I: moderate Solidly useful S: needs-investigation This issue must be examined or reproduced S: ready-for-design This issue needs design work laying out how it should be implemented C: ci Continuous integration T: bug A problem in an existing feature S: wont-fix This direction won't be taken in favor of another approach and removed S: needs-investigation This issue must be examined or reproduced S: ready-for-design This issue needs design work laying out how it should be implemented labels Sep 26, 2021
@alice-i-cecile
Copy link
Contributor Author

Could not get working: rust-lang/mdBook#394 no longer seems to work for me as dependencies cannot be found.

This can be re-opened if someone figures out how to get it working.

@alice-i-cecile alice-i-cecile added S: blocked-upstream No progress can be made on this until an upstream dependency fixes something I: polish Nits and aesthetics and removed T: bug A problem in an existing feature S: wont-fix This direction won't be taken in favor of another approach I: moderate Solidly useful labels Sep 27, 2021
@alice-i-cecile
Copy link
Contributor Author

alice-i-cecile commented Sep 27, 2021

Looks like a bug in upstream. Leaving this open with low importance; if anyone feels like tracking down and fixing this issue for upstream I'd be happy to add this functionality and suspect they'd be very grateful.

@alice-i-cecile alice-i-cecile added D: high Quite complex or very technically challenging I: low Nice to have T: enhancement A new feature and removed D: moderate Some thought and experience needed I: polish Nits and aesthetics labels Sep 27, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
C: ci Continuous integration D: high Quite complex or very technically challenging I: low Nice to have S: blocked-upstream No progress can be made on this until an upstream dependency fixes something T: enhancement A new feature
Projects
None yet
Development

No branches or pull requests

1 participant