Versions Compared

Key

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

...

...

See Trading reviews on Pull Requests for how to get immediate attention to that PR!

Notes

Next developers meeting

  • There will be no meeting on August 15, 2024
  • The next meeting will be held on August 22, 2024

Review of ongoing discussion topics

  • 9.0 development priorities
    • There are currently no news from the Steering Group regarding 9.0 development prioritization
    • A subgroup of Steering (9.0 Planning Group) is discussing recommendations and will pass these on to Steering at the end of this week
    • As a reminder, Steering tends to set general development priorities and does not provide a list of specific features (compare the Priority List for 8.0)
  • Optimization of workflow around code reviews and pull request
    • A previous discussion about this resulted in the creation of this wiki page: Incentivizing Code Reviews and PR Testing
      • Reviewed some of the main ways to speed up workflow listed on this page (trading pull requests, Pull preview as a service, GitPod)
      • All developers are encouraged to review this page; if anyone has additional ideas, please add them to the page or send them to Tim
      • If anyone has tried suggestions listed on this page, please share your notes as to your experience with the suggested solution
  • High CPU with DSpace 7 SSR
    • Looking for more volunteers to tackle this issue
  • Using both public/private REST API URLs for better performance
    • Has been assigned to 4Science
    • Currently no updates on this ticket
  • Simplifying the installation process
    • This is a larger task which will likely require several smaller fixes
    • Smaller, step-by-step improvements are welcome to move this along

Changes Group2Group Cache computation (#9760)

  • Connected to performance-related discussion
  • Could be a promising performance fix if implemented
  • Changed to high-priority

Review of 9.0 Project Board

  • General reminders
    • We need developers to review or test what is on the board (category "needs reviewers assigned" or "under review")
    • The main purpose of assignments is so that someone is tasked; others can review even if not assigned to a specific ticket
    • A number of items on this board have merge conflicts and need attention from developers.
  • "Reviewer approved" column
    • Tim will take a look at this column today or tomorrow and see if he can get some of these merged
    • Other Committers are encouraged to keep the number of items in this column as small as possible
  • "Needs Reviewer Assignment" column
    • Reviewed, discussed and assigned several tickets in this column
      • #8932 – Needs further discussion; will be discussed at a future developers meeting
      • #3078 – Assigned to Art (with backend ticket #9610)
      • #3076 – Assigned to Kim (with backend ticket #9608)
      • #9650 – Assigned to Pascal
      • #2971 – Assigned to Tim (with backend ticket #9489)
      • #3026 – Assigned to Art (with backend ticket #9562)
      • #3035 – Assigned to Art
      • #3054 – Assigned to Tim
      • #3065 – Assigned to Tim
      • #3075 – Assigned to Art
      • #3140 – may be replaced by #3199 if #3199 is approved