-
Notifications
You must be signed in to change notification settings - Fork 18
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
Add contribution guidance for Tools page #153
Add contribution guidance for Tools page #153
Conversation
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.
A couple of comments but in general this looks like a good update
Do we need to add a disclaimer saying that whilst we believe these tools are useful, we have not comprehensively quality assured or security checked them all and are not liable should any issues arise |
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.
Reviewed disclaimer wording on using the tools.
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.
Changes requested have been implemented or an issue created to discuss. Happy for this to now be merged
Aiming to get more user input on tools that are useful for reproducible analysis. Adding clear guidance on how to contribute to this page.