# 2022-11-11 Nix team meeting minutes #7

**URL:** https://discourse.nixos.org/t/2022-11-11-nix-team-meeting-minutes-7/23451
**Category:** Nix
**Created:** [November 24, 2022, 10:43am UTC](https://discourse.nixos.org/t/2022-11-11-nix-team-meeting-minutes-7/23451 "2022-11-24T10:43:20Z")
**Posts on this page:** 1
**Page:** 1

<div class="post-metadata">

### Author: ![fricklerhandwerk](https://discourse.nixos.org/user_avatar/discourse.nixos.org/fricklerhandwerk/32/2029_2.png) [@fricklerhandwerk](https://discourse.nixos.org/u/fricklerhandwerk)
#### Post date: [November 24, 2022, 10:43am UTC](https://discourse.nixos.org/t/2022-11-11-nix-team-meeting-minutes-7/23451/1 "2022-11-24T10:43:20Z")

</div>

Attendees: @edolstra @tomberek @thufschmitt @fricklerhandwerk @roberth @Ericson2314

# Planning discussion

- @fricklerhandwerk invited Elton Vecchietti from Modus Create to pitch his workshop at the [Nix hackathon in Zürich on 2022-11-26](https://www.meetup.com/nixos-user-group-zurich/events/289378125/)
  - will meet next Monday with @fricklerhandwerk, @edolstra, and whoever else is interested

- @fricklerhandwerk: Schedule a second meeting per week to have one for work and one for discussion (as @tomberek proposed)
  - @tomberek: discussion meetings should produce an ordered list of tasks to work on in the work meeting without the need for further discussion
    - it was effective for cleaning up little things
      - not necessarily good to deal with massive changes

  - what should we focus on for the work meetings? small uncontroversial things?
    - @tomberek: at least in the beginning, yes

  - when do we schedule the second meeting?
    - @tomberek: will open a [when2meet](https://www.when2meet.com/)

- @fricklerhandwerk define a process for deciding on an agenda
  - proposal:
    - have a file in the Nix repo
    - make a PR each week
    - add comments with suggestions for new items
    - who takes care of opening and closing the PR?

  - @tomberek: like the PR workflow
  - @edolstra: would like to have as little process as possible
    - why not add items to this document?
      - no way to comment

    - PR workflow looks good though
      - @fricklerhandwerk: have to determine who does the chores
        - should be the team lead
        - @edolstra: yes, will open a PR every week
        - @fricklerhandwerk: will continue publishing the meeting notes

- @fricklerhandwerk: fefine a process how we pick PRs for discussion
  - proposal:
    - every meeting spend 10-15 min to get an impression of the big picture first
      - next release milestone
      - skim oldest PRs
      - sort by 👍 or ❤
      - filter by topics we may care about

    - maintainers can add PRs they want to discuss to the project board directly
    - pick one obvious candidate from each of the above categories every week
    - in the mid-term establish some triaging process, e.g.:
      - team members commit 1h every week
      - label issues and PRs
      - ping authors of stale PRs
      - …

  - @tomberek: will set up a team handbook in Nix repo including the two meetings and some process

# Pull Request Review

- [Remove repeat option by lheckemann · Pull Request #7099 · NixOS/nix · GitHub](https://github.com/NixOS/nix/pull/7099)
  - assigned for merging

- [Auto-assign reviewers by file by Mic92 · Pull Request #7229 · NixOS/nix · GitHub](https://github.com/NixOS/nix/pull/7229#issuecomment-1311660746)
  - doesn’t work, see: [Test auto-assignment of reviewers by edolstra · Pull Request #7293 · NixOS/nix · GitHub](https://github.com/NixOS/nix/pull/7293)

- [https://github.com/NixOS/nix/pull/7278](https://github.com/NixOS/nix/pull/7278)
  - in-review, assigned for merging

- [add removing users to uninstall instructions by fricklerhandwerk · Pull Request #7279 · NixOS/nix · GitHub](https://github.com/NixOS/nix/pull/7279#pullrequestreview-1177310895)
  - merged

- [add CODEOWNERS by fricklerhandwerk · Pull Request #7277 · NixOS/nix · GitHub](https://github.com/NixOS/nix/pull/7277#issuecomment-1326262171)
  - on hold, @edolstra will first try to get [Auto-assign reviewers by file by Mic92 · Pull Request #7229 · NixOS/nix · GitHub](https://github.com/NixOS/nix/pull/7229) to work

- [\_\_allowNetwork when sandbox==relaxed by jcumming · Pull Request #7109 · NixOS/nix · GitHub](https://github.com/NixOS/nix/pull/7109#issuecomment-1311642193)
  - this may be a use case for impure derivations