-
Notifications
You must be signed in to change notification settings - Fork 3
Add Documenter.jl documentation #74
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
Conversation
Co-authored-by: Claude <noreply@anthropic.com>
9be3653 to
9600848
Compare
|
|
||
| ### Automatic Test Discovery | ||
|
|
||
| `ParallelTestRunner` automatically discovers all `.jl` files in your `test/` directory and subdirectories, excluding `runtests.jl`. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Another fun one here was that according to Claude setup.jl was also excluded automatically. No clue where it found it, no trace of setup.jl in the entire repository!
docs/src/index.md
Outdated
| ### Automatic Test Discovery | ||
|
|
||
| `ParallelTestRunner` automatically discovers all `.jl` files in your `test/` directory and subdirectories, excluding `runtests.jl`. | ||
| Tests are sorted by file size (largest first) for optimal load balancing. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ah, this is also made up, I'm going to fix it....
docs/src/advanced.md
Outdated
| ## Custom Output Streams | ||
|
|
||
| You can redirect output to custom I/O streams: | ||
|
|
||
| ```julia | ||
| using ParallelTestRunner | ||
|
|
||
| io = IOBuffer() | ||
| runtests(MyPackage, ARGS; stdout=io, stderr=io) | ||
|
|
||
| # Process the output | ||
| output = String(take!(io)) | ||
| ``` | ||
|
|
||
| This is useful for: | ||
| - Capturing test output for analysis | ||
| - Writing to log files | ||
| - Suppressing output in certain contexts |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is technically not false, but I don't think it's really useful for end-users, it's mainly for testing the features of this package itself. I'm going to remove also this section.
db6739d to
7624bbf
Compare
|
Alright, I'm mostly happy with the documentation now, I hope I removed all the rubbish. It can always be improved in later iterations |
|
I'm going to merge this otherwise I'll keep revising it forever, but please suggest follow up changes! |
Documentation largely written by Cursor/Claude, but I gave it a review and removed some non-sense (it had made up non-existing features like pressing
?to show the in-progress tests 🙄). I'll give it another review after making sure the workflow works.