Members attending: @edolstra @roberth @thufschmitt @fricklerhandwerk
Agenda
-
observations:
- Assigned column is getting too long
- go through those PRs first to get them closer to merging
- To discuss is the longest column, almost exclusively new features
- re-prioritise to focus on what we find most important
- @thufschmitt: for the discussion slot we should prioritise the following because it’s a yes-no-question:
- Assigned column is getting too long
-
@fricklerhandwerk: would be great if everyone on the team could go off with their assigned PRs and merge them when ready without further involvement of the team
- @edolstra: agree. was not aware that not everyone on the team has merge access.
- @thufschmitt: who has permissions to grant merge access?
- renamed Nix committers to Nix team
- deleted the Nix core team which is defunct since 2020
- added @roberth and @tomberek to Nix team, where all members have merge access
- @edolstra: agree. was not aware that not everyone on the team has merge access.
-
@fricklerhandwerk: would like us to focus again a bit more on process and move forward making our goals and routines more explicit
- @edolstra: should not fill the whole meeting with it, though
- @fricklerhandwerk: yes, must balance the meta discussion with other duties. still, should take time setting a course more deliberately. writing down anything substantial at all is already helpful.
- @roberth: one way to get more productive is to become more independent
- @fricklerhandwerk: agreed, we discussed the relevant considerations in the first meetings.
-
@roberth: would like to put the flakes schema design back on the agenda, specifically the
systemattribute -
@fricklerhandwerk: inviting pull request authors for next work session needs someone to take care of
- so far we predominantly review team member’s PRs
- prosed lead time: ~2 weeks
- so far we predominantly review team member’s PRs
Re-check assigned PRs
Discussed in Nix team meeting 2023-01-02:
-
define "store derivation" by fricklerhandwerk · Pull Request #7485 · NixOS/nix · GitHub
- merged
-
- asigned to @thufschmitt to merge after discussed fix-ups are pushed
-
- merged
-
Read file type by alex-ameen · Pull Request #7447 · NixOS/nix · GitHub
- still assigned to @roberth
-
add labeler workflow by Artturin · Pull Request #6338 · NixOS/nix · GitHub
- decision: delegate developing a process, merge the GitHub workflow
-
https://github.com/NixOS/nix/pull/7278
- merged
-
manual: architecture overview by fricklerhandwerk · Pull Request #7066 · NixOS/nix · GitHub
- merged
-
https://github.com/NixOS/nix/pull/7087#issuecomment-1369637978
- attribute should be prefixed with
unsafe
- attribute should be prefixed with