Skip to content

Pull Request Checklist

dondi edited this page May 5, 2026 · 16 revisions

For each pull request, the following checklist will be filled out before merging the pull request. Anything not passing will be noted as a bug to be fixed before the pull request is merged.

Pull Request Checklist

  • GitHub Actions C.I. build passes
  • All five Demos look as expected when loaded
  • Reload works as expected
  • Load an Excel file (GRN and PPI)
  • Load a SIF file (GRN and PPI)
  • Load a GraphML file (GRN)
  • Loading file with Error brings up error modal, and it looks as expected
  • Loading file with Warning brings up warnings modal
  • Loading a network from the database looks as expected when loaded (GRN and PPI)
  • Network can be exported to SIF (GRN and PPI) and GraphML (GRN-only)
  • Network, Expression data, and Additional Sheets can be exported to Excel
  • Image can be exported to PNG, SVG and PDF
  • Print works as expected
  • Restrict graph to viewport works as expected (check/uncheck)
  • Viewport Size Changing works as expected (small/medium/large/fit)
  • Toggle between Grid Layout and Force Graph Layout works as expected
  • Force Graph Parameter Sliders change the number above them and have an effect on the graph
  • Locking/Unlocking/Resetting/Undo Resetting the force graph parameters works as expected
  • Enabling and disabling node coloring works as expected
  • Node coloring options work as expected (selection top/bottom dataset, averaging values, changing max value)
  • Weighted graph loaded with the "Enable Edge Coloring" option checked appears as expected
  • Hide/Show Weights works as expected (always/never/upon mouseover)
  • Edge Weight Normalization Factor can be changed (set/reset)
  • Gray Edge Threshold can be changed, slider changes the number and has an effect on the graph
  • Checking Show Gray Edges as Dashed works as expected (check/uncheck)
  • D-pad left/right/top/bottom/center works as expected
  • Zoom slider changes number and has effect on graph (viewport and menu)
  • Right click on a node opens gene page, page is populated with correct data (currently broken, though)

To copy it into the pull request, copy this:

Pull Request Checklist
- [ ] GitHub Actions C.I. build passes
- [ ] All five Demos look as expected when loaded
- [ ] Reload works as expected
- [ ] Load an Excel file (GRN and PPI)
- [ ] Load a SIF file (GRN and PPI)
- [ ] Load a GraphML file (GRN)
- [ ] Loading file with Error brings up error modal, and it looks as expected
- [ ] Loading file with Warning brings up warnings modal
- [ ] Loading a network from the database looks as expected when loaded (GRN and PPI)
- [ ] Network can be exported to SIF (GRN and PPI) and GraphML (GRN-only)
- [ ] Network, Expression data, and Additional Sheets can be exported to Excel
- [ ] Image can be exported to PNG, SVG and PDF
- [ ] Print works as expected
- [ ] Restrict graph to viewport works as expected (check/uncheck)
- [ ] Viewport Size Changing works as expected (small/medium/large/fit)
- [ ] Toggle between Grid Layout and Force Graph Layout works as expected
- [ ] Force Graph Parameter Sliders change the number above them and have an effect on the graph
- [ ] Locking/Unlocking/Resetting/Undo Resetting the force graph parameters works as expected
- [ ] Enabling and disabling node coloring works as expected
- [ ] Node coloring options work as expected (selection top/bottom dataset, averaging values, changing max value)
- [ ] Weighted graph loaded with the "Enable Edge Coloring" option checked appears as expected
- [ ] Hide/Show Weights works as expected (always/never/upon mouseover)
- [ ] Edge Weight Normalization Factor can be changed (set/reset)
- [ ] Gray Edge Threshold can be changed, slider changes the number and has an effect on the graph
- [ ] Checking Show Gray Edges as Dashed works as expected (check/uncheck)
- [ ] D-pad left/right/top/bottom/center works as expected
- [ ] Zoom slider changes number and has effect on graph (viewport and menu)
- [ ] Right click on a node opens gene page, page is populated with correct data (currently broken, though)

Clone this wiki locally