about summary refs log tree commit diff
AgeCommit message (Collapse)AuthorFilesLines
2021-12-20 r/3322 fix(wpcarro/marcus): Add wpcarro to networkmanagerWilliam Carroll1-0/+1
This allows me to (e.g.) connect to wifi without being root. Change-Id: Iba2e8516aa7a9d24e52d7afa0de0c4f31e39be16 Reviewed-on: https://cl.tvl.fyi/c/depot/+/4421 Tested-by: BuildkiteCI Reviewed-by: wpcarro <wpcarro@gmail.com>
2021-12-20 r/3321 feat(wpcarro/marcus): Install pavucontrolWilliam Carroll1-0/+1
On the fifth day, there was sound. Change-Id: Idf0d3f06afa45311c360663da42fb3ed355344bd Reviewed-on: https://cl.tvl.fyi/c/depot/+/4420 Tested-by: BuildkiteCI Reviewed-by: wpcarro <wpcarro@gmail.com>
2021-12-20 r/3320 feat(wpcarro/marcus): Set wheelNeedsPassword = falseWilliam Carroll1-0/+2
Removing some keystrokes from my life at the cost of security. Thoughts, comments, and concerns are welcome! Change-Id: I04d877021c4ce15a4d432a6a73e38e60878521aa Reviewed-on: https://cl.tvl.fyi/c/depot/+/4419 Tested-by: BuildkiteCI Reviewed-by: wpcarro <wpcarro@gmail.com>
2021-12-20 r/3319 feat(wpcarro/marcus): Install telegramWilliam Carroll1-0/+1
More messaging clients :wahoo: Change-Id: I005f6ba0514af60b58ee0a33357b5ad60660a49c Reviewed-on: https://cl.tvl.fyi/c/depot/+/4418 Tested-by: BuildkiteCI Reviewed-by: wpcarro <wpcarro@gmail.com>
2021-12-20 r/3318 fix(wpcarro/emacs): Ensure laptop battery shows in modelineWilliam Carroll2-4/+11
TL;DR: - support marcus and diogenes in device.el - define and consume device-laptop? fn Change-Id: Ia6e1c2666cbd668466ef6b64a7c274d36b79d699 Reviewed-on: https://cl.tvl.fyi/c/depot/+/4392 Tested-by: BuildkiteCI Reviewed-by: wpcarro <wpcarro@gmail.com>
2021-12-20 r/3317 feat(grfn/bbbg): Add robots.txtGriffin Smith2-0/+4
This'll be publicly hosted, and I don't want search engines knowing about it. Change-Id: I801217d11f6656b254698b852e5dfb4d96305a38 Reviewed-on: https://cl.tvl.fyi/c/depot/+/4511 Reviewed-by: grfn <grfn@gws.fyi> Autosubmit: grfn <grfn@gws.fyi> Tested-by: BuildkiteCI
2021-12-20 r/3316 feat(grfn/bbbg): Add a skeleton event pageGriffin Smith3-4/+55
Change-Id: Id056a7d5bf21c34317957a75e7a91ed42f55a939 Reviewed-on: https://cl.tvl.fyi/c/depot/+/4510 Reviewed-by: grfn <grfn@gws.fyi> Autosubmit: grfn <grfn@gws.fyi> Tested-by: BuildkiteCI
2021-12-20 r/3315 feat(grfn/bbbg): Move nav to page-globalGriffin Smith3-27/+39
Change-Id: I0a313d9df818fcbfeea5484123c1e7bcec439d66 Reviewed-on: https://cl.tvl.fyi/c/depot/+/4509 Reviewed-by: grfn <grfn@gws.fyi> Autosubmit: grfn <grfn@gws.fyi> Tested-by: BuildkiteCI
2021-12-20 r/3314 feat(grfn/bbbg): Add attendee checksGriffin Smith13-7/+333
Change-Id: I7f96597ab3f0552cdecd0abac1ef50a68d3e0b7b Reviewed-on: https://cl.tvl.fyi/c/depot/+/4508 Tested-by: BuildkiteCI Reviewed-by: grfn <grfn@gws.fyi> Autosubmit: grfn <grfn@gws.fyi>
2021-12-19 r/3313 feat(ops/pipelines): annotate patchset builds with Gerrit URLsVincent Ambo1-0/+6
If available, provide a link back to Gerrit on the overview page of a build. Uses the default style (i.e. style unset), which makes it non-intrusive visually. Change-Id: I4271d589d548015b75762fd0584f3958bfcc53e5 Reviewed-on: https://cl.tvl.fyi/c/depot/+/4442 Tested-by: BuildkiteCI Reviewed-by: grfn <grfn@gws.fyi>
2021-12-19 r/3312 feat(grfn/bbbg): Display + allow editing attendee notesGriffin Smith1-16/+44
Change-Id: I75353d64651a19beb44d3726d2645608bac0cde5 Reviewed-on: https://cl.tvl.fyi/c/depot/+/4505 Reviewed-by: grfn <grfn@gws.fyi> Autosubmit: grfn <grfn@gws.fyi> Tested-by: BuildkiteCI
2021-12-19 r/3311 style: format all Go codeVincent Ambo5-46/+46
The code in //users/wpcarro/tools/monzo_ynab/ynab/client.go was not valid Go and has been commented out. Change-Id: Icb4003607f30294dcbf60132eb7722702c7f0d84 Reviewed-on: https://cl.tvl.fyi/c/depot/+/4400 Tested-by: BuildkiteCI Reviewed-by: wpcarro <wpcarro@gmail.com> Reviewed-by: Profpatsch <mail@profpatsch.de>
2021-12-19 r/3310 fix(grfn/emacs): Clean up init.elGriffin Smith1-63/+2
Change-Id: I7b4d9f6b4f008ed432f3628c18e41243f9f4a260 Reviewed-on: https://cl.tvl.fyi/c/depot/+/4503 Reviewed-by: grfn <grfn@gws.fyi> Autosubmit: grfn <grfn@gws.fyi> Tested-by: BuildkiteCI
2021-12-19 r/3309 feat(grfn/bbbg): Add an attendee search formGriffin Smith3-16/+30
Change-Id: I6f4d5102c1d3a36cb706a637196a2053739003bb Reviewed-on: https://cl.tvl.fyi/c/depot/+/4502 Reviewed-by: grfn <grfn@gws.fyi> Autosubmit: grfn <grfn@gws.fyi> Tested-by: BuildkiteCI
2021-12-19 r/3308 feat(grfn/bbbg): Add an attendees listGriffin Smith5-1/+73
Add a list of attendees visible only to authenticated users, with stats about rsvps, events attended, and no-shows. Change-Id: Ib9a0fe8acf8c616fb725c613494b37121a1ad0e4 Reviewed-on: https://cl.tvl.fyi/c/depot/+/4501 Reviewed-by: grfn <grfn@gws.fyi> Autosubmit: grfn <grfn@gws.fyi> Tested-by: BuildkiteCI
2021-12-19 r/3307 fix(ops/besadii): fix Gerrit URL format ... againVincent Ambo1-1/+1
got into some kind of race with different patchsets of this CL somehow, idk Change-Id: I3dcdb708f141829b866fbd786483710b43ea9824 Reviewed-on: https://cl.tvl.fyi/c/depot/+/4481 Autosubmit: tazjin <mail@tazj.in> Reviewed-by: grfn <grfn@gws.fyi> Tested-by: BuildkiteCI
2021-12-19 r/3306 fix(ops/besadii): Only set branch to CL when building patchsetsVincent Ambo1-3/+5
If we set this for canon, then stuff starts to fail in non-obvious ways. Change-Id: I3bf38e29151c6066aaf4eba68ae387272d8a82c2 Reviewed-on: https://cl.tvl.fyi/c/depot/+/4463 Tested-by: BuildkiteCI Autosubmit: tazjin <mail@tazj.in> Reviewed-by: grfn <grfn@gws.fyi>
2021-12-19 r/3305 fix(ops/besadii): Stop path.Join from eating our URLVincent Ambo1-1/+1
apparently this chomps away at things inside of fragment strings Change-Id: Ie60d52d101dc4281b3a62c228af076791e1c7928 Reviewed-on: https://cl.tvl.fyi/c/depot/+/4462 Tested-by: BuildkiteCI Reviewed-by: grfn <grfn@gws.fyi>
2021-12-19 r/3304 feat(ops/besadii): Pass Gerrit link to builds as an envvarVincent Ambo1-1/+7
This makes it possible to annotate builds with a link back to Gerrit. Change-Id: If351785d3b631b96753d41f417ca94bc7a95ac54 Reviewed-on: https://cl.tvl.fyi/c/depot/+/4441 Reviewed-by: grfn <grfn@gws.fyi> Tested-by: BuildkiteCI
2021-12-19 r/3303 feat(ops/besadii): Make branch key cl/XXXXGriffin Smith1-1/+5
The branch key for buildkite builds isn't actually used to fetch if a commit is given - instead, it's just a visual grouping of multiple builds. This means we can just make the branch key cl/<cl number>, which is the convention we already use to refer to CLs and gets us a nice visual grouping of builds of successive patchsets of the same CL number, even though the ref we're providing isn't a real ref. Change-Id: Iaa9111297a88f965fda94cd8266240106f58a100 Reviewed-on: https://cl.tvl.fyi/c/depot/+/4347 Tested-by: BuildkiteCI Reviewed-by: tazjin <mail@tazj.in> Autosubmit: grfn <grfn@gws.fyi>
2021-12-19 r/3302 feat(whitby): Add buildkite agents to docker groupGriffin Smith1-1/+1
I'd like to be able to run extra CI steps that include running docker containers (to integration test things like webapps that connect to a database). To do this the buildkite agents themselves need permission to do docker things. Change-Id: I3c9a488708f0e12a508754ac41f04148ca7aedac Reviewed-on: https://cl.tvl.fyi/c/depot/+/4408 Tested-by: BuildkiteCI Autosubmit: grfn <grfn@gws.fyi> Reviewed-by: tazjin <mail@tazj.in>
2021-12-19 r/3301 feat(grfn/bbbg): Require auth for a bunch of stuffGriffin Smith4-14/+29
Only authenticated users should be allowed to create events, etc. Change-Id: I1f560365dbf583cf6cecf4a0798952f323bbd42e Reviewed-on: https://cl.tvl.fyi/c/depot/+/4412 Reviewed-by: grfn <grfn@gws.fyi> Autosubmit: grfn <grfn@gws.fyi> Tested-by: BuildkiteCI
2021-12-19 r/3300 feat(grfn/bbbg): Add a Sign Out buttonGriffin Smith2-5/+21
Change-Id: Ia57a31101dc294dba54a4fde7af2b636288bd4cd Reviewed-on: https://cl.tvl.fyi/c/depot/+/4411 Reviewed-by: grfn <grfn@gws.fyi> Autosubmit: grfn <grfn@gws.fyi> Tested-by: BuildkiteCI
2021-12-19 r/3299 fix(grfn/bbbg): Properly handle already-existing userGriffin Smith2-1/+10
If a user already exists with the discord ID we just authenticated, just return it rather than trying to create another user. Change-Id: I582c4c8c55f6fb399295b3e691b0bc59e77151f4 Reviewed-on: https://cl.tvl.fyi/c/depot/+/4410 Reviewed-by: grfn <grfn@gws.fyi> Autosubmit: grfn <grfn@gws.fyi> Tested-by: BuildkiteCI
2021-12-19 r/3298 feat(grfn/bbbg): Allow Organizers to sign in via DiscordGriffin Smith10-18/+409
Allow users with the Organizers role to sign in via a Discord Oauth2 handshake, creating a user in the users table and adding the ID of that user to the session. Change-Id: I39d9e17433e71b07314b9eabb787fb9214289772 Reviewed-on: https://cl.tvl.fyi/c/depot/+/4409 Tested-by: BuildkiteCI Reviewed-by: grfn <grfn@gws.fyi> Autosubmit: grfn <grfn@gws.fyi>
2021-12-18 r/3297 fix(tazjin/emacs): add dwebp to Emacs' $PATHVincent Ambo1-1/+4
Change-Id: I20723b7d98a24e3f06810f0a0ede8a05c80156a8 Reviewed-on: https://cl.tvl.fyi/c/depot/+/4405 Reviewed-by: tazjin <mail@tazj.in> Autosubmit: tazjin <mail@tazj.in> Tested-by: BuildkiteCI
2021-12-18 r/3296 fix(3p/overlays): Attempt to fix hard-to-debug suspend issueVincent Ambo1-0/+10
I have an issue where my AMD laptop occasionally does not wake from suspend. The only thing ever indicated in the logs is related to the patched line here: xss-lock[2409537]: saver_blank: line 19: exec: sleep: not found This stems from xsecurelock calling a script that in turn calls sleep, in a loop. I don't know if this causes my unsuspend problem because this also happens sometimes without anything failing, but it's the best lead I've got for now. Will run with this fix for a while and see if anything changes in the stability of suspend. Change-Id: Ie8190433c2125cbeddb909013be9dfa00141d5c6 Reviewed-on: https://cl.tvl.fyi/c/depot/+/4403 Tested-by: BuildkiteCI Reviewed-by: sterni <sternenseemann@systemli.org>
2021-12-18 r/3295 feat(tools/depotfmt): add a treefmt wrapper with depot configurationVincent Ambo4-0/+37
Right now the only included formatter is gofmt, but we will extend this over time. The version of treefmt is bumped to 0.3.0 (which supports custom config files) until this lands in nixpkgs. Change-Id: I1e1aafd05ec7427c616f90c90490c528ecb2615c Reviewed-on: https://cl.tvl.fyi/c/depot/+/4399 Tested-by: BuildkiteCI Reviewed-by: grfn <grfn@gws.fyi>
2021-12-18 r/3294 fix(tools/magrathea): prevent trailing slash for // in pathsterni1-6/+7
All targets would have no trailing slash, so it was at least ugly that // had one as the only legal target. Change-Id: I1b60850ac86d8c550f262841694fb00c518413b8 Reviewed-on: https://cl.tvl.fyi/c/depot/+/4404 Autosubmit: sterni <sternenseemann@systemli.org> Tested-by: BuildkiteCI Reviewed-by: tazjin <mail@tazj.in>
2021-12-18 r/3293 feat(tools/cheddar): shortlink depot revisionssterni2-0/+12
r/3000 will now be shortlinked to show the commit in cgit. Going via atward probably doesn't make sense at this point, since the depot refs are not available in sourcegraph at all (for reasons I can't repeat). Switching to atward might be interesting when/if we introduce support for shortlinking // paths. Fixes: b/163 Change-Id: I57c1a7d02d881e4f8b3ee1f71755dd7930925dc4 Reviewed-on: https://cl.tvl.fyi/c/depot/+/4402 Tested-by: BuildkiteCI Autosubmit: sterni <sternenseemann@systemli.org> Reviewed-by: tazjin <mail@tazj.in>
2021-12-18 r/3292 feat(tools/magrathea): let users override the repository rootVincent Ambo1-3/+4
for global installations of magrathea, setting MG_ROOT can be a way to switch quickly between different repositories (esp. in combination with `cd (mg path)`). Change-Id: I4627fe78b7cc112b75ab57e7806ffd85c6d38aee Reviewed-on: https://cl.tvl.fyi/c/depot/+/4396 Tested-by: BuildkiteCI Reviewed-by: wpcarro <wpcarro@gmail.com>
2021-12-18 r/3291 feat(tools/magrathea): add a path commandVincent Ambo1-4/+19
this command prints the absolute directory for a given target. it can be combined with shell aliases to add quick navigation commands. unfortunately due to the nature of computers implementing something like `mg cd` directly is not possible. Change-Id: Icc88eb97384812c620c49fe2de8fa331f4d7153b Reviewed-on: https://cl.tvl.fyi/c/depot/+/4395 Tested-by: BuildkiteCI Reviewed-by: wpcarro <wpcarro@gmail.com>
2021-12-18 r/3290 feat(tools/magrathea): wire up parsed targets with commandsVincent Ambo1-24/+61
magrathea now does what it says on the tin - build and shell commands can be used with the targets specified on the command line. implementation notes: * string representation of target has been changed to look like the target spec format, this is now used in user-facing messages * errors returned by the target parser make the program exit with status 1 * normalisation could be done better (for example, maybe it makes sense to always do it) but it's good enough for now Change-Id: Ib85f389a5cec92b3c2f3b9c0b40764435bbcc68b Reviewed-on: https://cl.tvl.fyi/c/depot/+/4394 Tested-by: BuildkiteCI Reviewed-by: wpcarro <wpcarro@gmail.com>
2021-12-18 r/3289 chore(3p/overlays): Remove unnecessary steam fixVincent Ambo1-5/+0
this is now upstreamed in nixpkgs Change-Id: I6131edd326fa347f546df30b865cbc2327ae2577 Reviewed-on: https://cl.tvl.fyi/c/depot/+/4398 Reviewed-by: tazjin <mail@tazj.in> Autosubmit: tazjin <mail@tazj.in> Tested-by: BuildkiteCI
2021-12-17 r/3288 fix(ops/diogenes): Ensure diogenes buildsWilliam Carroll4-6/+12
diogenes "passed" CI because the file was named configuration.nix (vestage from the NixOS default /etc/nixos/configuration). This CL fixes some issues I encountered after running depot/bin/rebuild-system. TL;DR: - rename configuration.nix -> default.nix to trigger CI - add diogenes to my systems - add public SSH key Change-Id: I24197b8936c201267db6f71f00099dce590eac1d Reviewed-on: https://cl.tvl.fyi/c/depot/+/4388 Tested-by: BuildkiteCI Reviewed-by: wpcarro <wpcarro@gmail.com> Reviewed-by: tazjin <mail@tazj.in> Autosubmit: wpcarro <wpcarro@gmail.com>
2021-12-17 r/3287 feat(tools/magrathea): add weirdly overengineered target spec parserVincent Ambo1-1/+105
i'm not sure what happened here, but it works (yes, the fancy target printing is completely unnecessary, but oh well): #;152> (parse-target "foo") #target(foo) #;153> (parse-target "//foo") #target(//foo) #;154> (parse-target "//foo/bar") #target(//foo/bar) #;155> (parse-target "//foo/bar/") #target(//foo/bar) #;156> (parse-target "//foo/bar:baz") #target(//foo/bar:baz) #;157> (parse-target "//foo/bar/:baz") #target(//foo/bar:baz) #;158> (parse-target "//foo/bar:") (error . "unexpected end of input while parsing virtual target") #;159> (parse-target "//foo//") (error . "unexpected root-anchor while parsing normal target") the most notable thing is that trailing slashes are allowed in the physical targets, since people may be autocompleting these on the shell from folder names. Change-Id: I32975ad77fe2a327130dc9574011fe92cce49f84 Reviewed-on: https://cl.tvl.fyi/c/depot/+/4393 Tested-by: BuildkiteCI Reviewed-by: grfn <grfn@gws.fyi> Reviewed-by: wpcarro <wpcarro@gmail.com>
2021-12-17 r/3286 feat(tools/magrathea): bootstrap a tool for working with depotVincent Ambo4-0/+137
this is going to be a serious version of //tools/depot-build. right now it doesn't support parsing any target specs yet, so only shells and builds for the physical project of the current folder work. Change-Id: I4308e29da940571622ff9e539fbb8ededd27aca7 Reviewed-on: https://cl.tvl.fyi/c/depot/+/4335 Tested-by: BuildkiteCI Reviewed-by: tazjin <mail@tazj.in> Reviewed-by: grfn <grfn@gws.fyi>
2021-12-17 r/3285 feat(nix/utils): handle toString-able sets produced by builtins.pathsterni2-3/+8
When preparing cl/4381 I noticed that we actually handle this case properly. depot.nix.utils.storePathName depot.path now works as expected. Change-Id: Ice9329c67b2e2210852012f5abe82fbbb13193de Reviewed-on: https://cl.tvl.fyi/c/depot/+/4382 Tested-by: BuildkiteCI Reviewed-by: tazjin <mail@tazj.in> Autosubmit: sterni <sternenseemann@systemli.org>
2021-12-17 r/3284 chore(tazjin/tverskoy): Increase / tmpfs size to 8GVincent Ambo1-1/+1
Any less and building 'awscli' is not possible ... Change-Id: I99105b767f4afa4cf0d072d57b08963bc12ff994 Reviewed-on: https://cl.tvl.fyi/c/depot/+/4391 Reviewed-by: tazjin <mail@tazj.in> Autosubmit: tazjin <mail@tazj.in> Tested-by: BuildkiteCI
2021-12-16 r/3283 feat(wpcarro/marcus): Prefer light to xbacklightWilliam Carroll2-9/+24
TL;DR: - Enable light (let there be light!) - Refactor screen-brightness.el TIL: `nixos-options programs.enable.light` outputs docs from the command-line Change-Id: Ib60a9b3c9b7f6f96fda29fce1a97ca9995822048 Reviewed-on: https://cl.tvl.fyi/c/depot/+/4385 Tested-by: BuildkiteCI Reviewed-by: wpcarro <wpcarro@gmail.com>
2021-12-16 r/3282 feat(wpcarro/marcus): Add some TVL goodiesWilliam Carroll1-17/+32
TL;DR: - enable TVL cache (ty @grfn) - enable automatic-gc (ty @tazjin) Change-Id: I2c7def718f10d90c6921461c6135f18358e33198 Reviewed-on: https://cl.tvl.fyi/c/depot/+/4362 Tested-by: BuildkiteCI Reviewed-by: wpcarro <wpcarro@gmail.com>
2021-12-16 r/3281 feat(wpcarro/diogenes): Add diogenes to the depotWilliam Carroll1-0/+56
Incoming! Change-Id: I12f9202fea8eb834d40de43109871aef2085416e Reviewed-on: https://cl.tvl.fyi/c/depot/+/4361 Tested-by: BuildkiteCI Reviewed-by: wpcarro <wpcarro@gmail.com>
2021-12-16 r/3280 feat(wpcarro/nixos): Define keys.nixWilliam Carroll1-0/+9
This opens the door for a whole bunch of neat features. Change-Id: I15066ca9d8fe38921ebc8c4e995bd282162a4577 Reviewed-on: https://cl.tvl.fyi/c/depot/+/4359 Tested-by: BuildkiteCI Reviewed-by: wpcarro <wpcarro@gmail.com>
2021-12-16 r/3279 refactor(wpcarro/nixos): Rename installer -> isoWilliam Carroll2-12/+17
Right now this is just a sketch for a larger idea of writing a binary called `create-installer` (see docs). Change-Id: Ib8e2acfd5d350ccc344329d87983c415f166d02b Reviewed-on: https://cl.tvl.fyi/c/depot/+/4358 Tested-by: BuildkiteCI Reviewed-by: wpcarro <wpcarro@gmail.com>
2021-12-16 r/3278 feat(wpcarro/emacs): Install tvl ElispWilliam Carroll2-0/+8
All I know is that this extends my magit configuration for Gerrit support. Change-Id: I35c9a264ff25ce6e08b7033e0e022d4047021f05 Reviewed-on: https://cl.tvl.fyi/c/depot/+/4357 Tested-by: BuildkiteCI Reviewed-by: wpcarro <wpcarro@gmail.com>
2021-12-16 r/3277 fix(wpcarro/emacs): Don't require evil-magitWilliam Carroll1-1/+0
It's part of `evil-collection` now :) Change-Id: Id69db08e538d10e46dc63c979d232aeb1a639f32 Reviewed-on: https://cl.tvl.fyi/c/depot/+/4356 Tested-by: BuildkiteCI Reviewed-by: wpcarro <wpcarro@gmail.com>
2021-12-16 r/3276 feat(wpcarro/emacs): Prefer doom-peacock themeWilliam Carroll1-1/+1
Tis the season for a dark theme Change-Id: I3a0354a3eaa4294e51f14e5af52a12be01e89300 Reviewed-on: https://cl.tvl.fyi/c/depot/+/4355 Tested-by: BuildkiteCI Reviewed-by: wpcarro <wpcarro@gmail.com>
2021-12-16 r/3275 feat(wpcarro/marcus): Install xsecurelockWilliam Carroll2-2/+5
TL;DR: - Install `xsecurelock` - Support screen-locking for personal (and corporate) devices Change-Id: Ife2ead109c60b69b817775d8d6524c6aece5f376 Reviewed-on: https://cl.tvl.fyi/c/depot/+/4354 Tested-by: BuildkiteCI Reviewed-by: wpcarro <wpcarro@gmail.com>
2021-12-16 r/3274 feat(wpcarro/marcus): Disable natural scrollingWilliam Carroll1-1/+1
Something about the end-result of this doesn't feel quite natural to me. Change-Id: Idaa7a991893d386110bd75f9288b4c3831a59cae Reviewed-on: https://cl.tvl.fyi/c/depot/+/4353 Tested-by: BuildkiteCI Reviewed-by: wpcarro <wpcarro@gmail.com>
2021-12-16 r/3273 chore(wpcarro/marcus): Format hardware.nixWilliam Carroll2-22/+17
Make some of the auto-generated code look a bit more like human-generated code. Change-Id: I07f543b66275dc8be9db6d8106ef6045541574d3 Reviewed-on: https://cl.tvl.fyi/c/depot/+/4352 Tested-by: BuildkiteCI Reviewed-by: wpcarro <wpcarro@gmail.com>
generated by cgit-pink 1.4.1 (git 2.44.2) at 2024-11-03T00·17+0000