2023-03-03 Nix team meeting minutes #37

Development Nix
1

Agenda

  • Triage (30 min)
  • Check stuck backport PRs
  • Enabling contributors; permissions (15-30 min)
  • Discussion (< 15 min)

Triage

Backport PRs

Contributors and permissions

Discussion

  • @Ericson2314: would like to bring up doing more work async between the meetings

    • propose to only bring up bigger issues in the whole group
    • e.g. 2+ team members review async and merge without guilt
      • Single reviewer is still fine
      • Have been instances of people approving but not feeling comfortable hitting the button – that’s not their fault!
      • “two people rule” is about getting over the hurdle without needing a synchronous meeting
        • with multiple people in effect jointly hitting “merge” together, no single person can be unfairly scape-goated
    • What would make people more comfortable reviewing more code?

  • @fricklerhandwerk: alternative, what we did so far, is dumping almost everything on the triage board and dispatch from there

    • @regnat: yes, especially for things that are uncontroversial it’s productive
    • @Ericson2314: we have lots of things in the backlog, and while we get through with new things, the backlog doesn’t really shrink
    • @fricklerhandwerk: if we focused more on making long-overdue design decisions, new PRs will more likely fall into place naturally, because people will find it easier to determine what to do
    • agreement: @Ericson2314 will make a write-up with proposal

  • Generate API docs with Doxygen by Ericson2314 · Pull Request #7896 · NixOS/nix · GitHub

    • decision: we really want this
      • add a separate makefile target and derivation output to avoid the impression this is a public interface
1 Like