-
Notifications
You must be signed in to change notification settings - Fork 62
Enhance the Tests section with best practices for clarity and documentation #509
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
Comments
I found that tests are already covered in this issue: #59 (comment) I'm realising now that the current "Tests" section already does a solid job of covering the basics. As a beginner, I don’t feel I can meaningfully contribute more at this stage, so I’ll step back and leave this issue open in case someone with more experience wants to build on it. I'm happy to revisit this in the future as I gain more insight. |
@all-contributors add @akhilkrishnar0 for ideas, review |
I've put up a pull request to add @akhilkrishnar0! 🎉 |
hey @akhilkrishnar0 I can add this issue to our help-wanted board. Could you provide some more details around what you'd like to see added to that section? I'm asking for this because it will make it easier for someone else to pick things up if we know what is missing!! Many thanks! |
Uh oh!
There was an error while loading. Please reload this page.
Hi everyone,
I'd like to propose adding a new “Tests” section to the packaging guide that outlines best practices for writing clear, well-documented tests. This idea came up in a recent conversation pyOpenSci/software-peer-review#201, where we discussed the value of making tests easier for reviewers and contributors to understand.
Here’s what the new section could include:
This would help both reviewers (who need to understand what’s being tested) and new contributors (who may want to add tests or debug).
Thanks to @ucodery for suggesting that I open this issue to get the conversation started.
Any comments or contributions are invited.
The text was updated successfully, but these errors were encountered: