Editing Reviewing Octave Forge packages
Jump to navigation
Jump to search
The edit can be undone. Please check the comparison below to verify that this is what you want to do, and then publish the changes below to finish undoing the edit.
Latest revision | Your text | ||
Line 1: | Line 1: | ||
{{ | {{Note|More than one person can review a package, in fact that would be great.}} | ||
The reviewing procedure in short: | The reviewing procedure in short: | ||
Line 5: | Line 5: | ||
# Pick a package at https://sourceforge.net/p/octave/package-releases/ | # Pick a package at https://sourceforge.net/p/octave/package-releases/ | ||
# Copy the check-list below and fill it out, marking each box: | # Copy the check-list below and fill it out, marking each box: | ||
#* <code>[x]</code> for | #* <code>[x]</code> for passed | ||
#* <code>[ ]</code> for | #* <code>[n/a]</code> for non-applicable | ||
#* <code>[ ]</code> for skipped (because you cannot do it, missing software, etc) | |||
#* <code>[F]</code> for fails. | |||
# Paste your filled in check-list as a comment on the issue above | # Paste your filled in check-list as a comment on the issue above | ||
# An [https://sourceforge.net/p/octave/_members/ Octave Forge Admin] will try to look over your review and hopefully release the package. | # An [https://sourceforge.net/p/octave/_members/ Octave Forge Admin] will try to look over your review and hopefully release the package. | ||
Line 13: | Line 15: | ||
<pre> | <pre> | ||
# Repository | # Repository | ||
- [ ] | - [ ] maintainer has specified a corresponding revision commit/changeset | ||
- [ ] | - [ ] checkout code from Sourceforge, commit/changeset is present | ||
# pkg-tool runs without errors or warnings | # Compile and Install | ||
- [ ] release candidate installs on latest stable Octave release | |||
- [ ] no compiler errors or warnings | |||
- [ ] ran tests using <code>runtests /path/to/pkg</code> (use <code>oruntests</code> instead in Octave 6.0.0 or later) | |||
- [ ] ran all tests, including those in src (how??) | |||
- [ ] ran doctest on all functions (optional) | |||
- [ ] ran generate_package_html (if Makefile present try <code> make html </code>) | |||
- [ ] no makeinfo errors and warnings during HTML build | |||
- [ ] unpacked and spot-checked the generated HTML documentation | |||
# `pkg`-tool runs without errors or warnings | |||
- [ ] `pkg load foo` | - [ ] `pkg load foo` | ||
- [ ] `pkg unload foo` | - [ ] `pkg unload foo` | ||
- [ ] `pkg uninstall foo` | - [ ] `pkg uninstall foo` | ||
- Above steps were run on Octave and OS versions: | - [ ] Above steps were run on Octave and OS versions: | ||
- ________ | |||
- ________ | |||
# Package files in release candidate tarball | # Package files in release candidate tarball | ||
- [ ] `DESCRIPTION` version | - [ ] `DESCRIPTION` version updated | ||
- [ ] `DESCRIPTION` minimum Octave tested | - [ ] `DESCRIPTION` minimum Octave tested | ||
- [ ] `DESCRIPTION` all dependencies are listed | - [ ] `DESCRIPTION` all dependencies are listed | ||
Line 49: | Line 55: | ||
</pre> | </pre> | ||
[[Category:Octave Forge]] | |||