| 20 Mar 2024 |
 infinisil | We could do something like "For every master push, check if its version number increased, if yes, create a release" | 23:23:47 |
 Philip Taron (UTC-8) | Having that in the git metadata is quite nice. | 23:24:09 |
| Philip Taron (UTC-8) | Sure, it's more permissions, but it's really nice. | 23:24:28 |
| infinisil | I guess we really don't need a separate machine account for the automated releases | 23:25:43 |
| infinisil | We can just give the github action permission for that | 23:26:01 |
| infinisil | The separate machine account workaround is only needed to trigger workflows from PRs triggered by workflows | 23:26:30 |
| Philip Taron (UTC-8) | In reply to @infinisil:matrix.org
Would be nicer if it would automatically do a weekly PR, which when merged creates a release
The separate machine account workaround is only needed to trigger workflows from PRs triggered by workflows
I'm getting confused.
We do want an automatic-ish update of dependencies. ✅ -- but with an explicit human in the loop hitting the merge button.
We do want an automatic-ish release process. ✅ -- but with an explicit human in the loop to move from draft to public.
So, we want a separate machine account, so that the CI all fits together, and doesn't get wedged like #14 did.
Right?
| 23:34:24 |
| infinisil | Ahh right! | 23:34:53 |
| infinisil | I'm getting confused too 😅 | 23:34:59 |
| infinisil | Philip Taron (UTC-8): Concretely, how about this:
- The version gets bumped (probably in
Cargo.toml) with a PR
- This can either happen manually if somebody remembers
- Otherwise a bot will automatically do that with a PR that just does the version bump
- A separate workflow, triggered on main branch pushes, takes care to do the release whenever the version gets bumped
| 23:39:22 |
| infinisil | * Philip Taron (UTC-8): Concretely, how about this:
-
The version gets bumped (probably in Cargo.toml) with a PR
- This can either happen manually if somebody remembers
- Otherwise a bot will automatically do that with a PR that just does the version bump
-
A separate workflow, triggered on main branch pushes, takes care to do the release whenever the version gets bumped
| 23:39:39 |
| infinisil | For 1. we need the separate machine account, but it doesn't need any permissions because it just creates a PR. For 2. we don't need a separate machine account and can just give the workflow the permission to push a tag/release | 23:40:37 |
| infinisil | * Philip Taron (UTC-8): Concretely, how about this:
- The version gets bumped (probably in
Cargo.toml) with a PR
- This can either happen manually if somebody remembers
- Otherwise a bot will automatically do that regularly with a PR that just does the version bump
- A separate workflow, triggered on main branch pushes, takes care to do the release whenever the version gets bumped
| 23:42:39 |
| infinisil | So yes, separate machine account needed for both release bump PRs and dependency bump PRs | 23:43:28 |
| infinisil | Maybe that should be the same PR actually | 23:43:40 |
|  @federicodschonborn:matrix.org joined the room. | 23:48:29 |
| Philip Taron (UTC-8) | In reply to @infinisil:matrix.org
Maybe that should be the same PR actually That's a fine idea. | 23:49:04 |
| 21 Mar 2024 |
| infinisil | Philip Taron (UTC-8): Just opened https://github.com/NixOS/nixpkgs-check-by-name/pull/15 to hopefully fix the automated updates | 00:40:38 |
|  Rhys joined the room. | 02:19:45 |
| infinisil | And automated releases: https://github.com/NixOS/nixpkgs-check-by-name/pull/16 :) | 03:37:53 |
| infinisil | Bed time for me now though o/ | 03:38:06 |
|  @aloisw:kde.org joined the room. | 08:43:47 |
|  NixOS Moderation Botchanged room power levels. | 18:03:15 |
|  @grahamc:nixos.org left the room. | 20:10:23 |
| 22 Mar 2024 |
| infinisil | The move of nixpkgs-check-by-name to the new repository is almost complete now: https://github.com/NixOS/nixpkgs/pull/297901 :) | 01:38:16 |
| infinisil | Oh idea: Automatically create a PR to update the pinned tooling in Nixpkgs when an automated release happens | 01:41:47 |
| infinisil | That would be nice.. | 01:41:50 |
| 23 Mar 2024 |
|  SomeoneSerge (back on matrix) changed their display name from SomeoneSerge (hash-versioned python modules when) to SomeoneSerge (migrating synapse). | 02:10:51 |
| 25 Mar 2024 |
 willbush | In reply to @infinisil:matrix.org
The move of nixpkgs-check-by-name to the new repository is almost complete now: https://github.com/NixOS/nixpkgs/pull/297901 :) Are you hosting office hours today? | 17:35:07 |
| infinisil | In reply to @willbush:matrix.org
Are you hosting office hours today? Yup :) | 17:35:31 |
