Versions Compared

Key

  • This line was added.
  • This line was removed.
  • Formatting was changed.

...

  • New Developers Hub - Draft docs for new developers started by Hardy Pottinger 
  • Pull Request "Trading" (discussed/approved in 2023-08-24 DSpace Developers Meeting)
    • Developers are encouraged to ask other developer(s) to review/test their PR in exchange for reviewing/testing a PR created by the other developer.  This allows both developers to get more immediate feedback!
    • Tim Donohue  will also take part in this PR "trading" but in a more general fashion.
      1. If you review or test any two similar-sized PRs (of your choice), I (Tim) will promise to review one of your PRs as soon as possible (Your PR go to the top of my "to do" list.)
        1. PRs you review or test can be any PR on one of our boards (7.6.1 Board or 8.0 Board), provided they are similar-size to your own PR.
        2. PRs you review or test must be from a developer at a different institution from your own.
        3. You must submit useful feedback on the PR you've reviewed/tested (via a comment on the PR or similar).  It can be positive or negative feedback (if you test it and it doesn't work for you, that still counts).
      2. If I (Tim) don't notice your two reviews, please message me (privately is fine) via Slack or email & let me know which of your PRs you want me to review as soon as possible.
      3. You can also "trade" to support someone else's PR.   Review any two PRs and then ask me (Tim) to review someone else's similar-sized PR.

Resources for making Code Reviews / Testing easier

...