specify _class module attributes to explicitly declare module types #398

Merged
kiara merged 1 commit from kiara/fediversity:declare-classes into main 2025-06-23 17:24:55 +02:00
Owner

closes #93.

note that this includes classes:

  • nixos
  • nixosTest
  • nixops4Resource
  • nixops4Deployment

.. and my (made-up, as per the docs):

  • nix-unit
  • package

.. while i did not manage to cover:

  • service tests, given pkgs.nixosTest seemed to not actually like _class = "nixosTest" (?!)

... nor #93's mentioned destructured arguments for that matter, as per Fediversity/Fediversity#93 (comment) - let me know if that is still desired as well.

closes #93. note that this includes classes: - `nixos` - `nixosTest` - `nixops4Resource` - `nixops4Deployment` .. and my (made-up, as per the [docs](https://ryantm.github.io/nixpkgs/module-system/module-system/#module-system-lib-evalModules-param-class)): - `nix-unit` - `package` .. while i did not manage to cover: - service tests, given `pkgs.nixosTest` seemed to not actually like `_class = "nixosTest"` (?!) ... nor #93's mentioned destructured arguments for that matter, as per https://git.fediversity.eu/Fediversity/Fediversity/issues/93#issuecomment-8045 - let me know if that is still desired as well.
kiara force-pushed declare-classes from c9b4301c70
Some checks failed
/ check-pre-commit (pull_request) Successful in 13s
/ check-data-model (pull_request) Successful in 19s
/ check-peertube (pull_request) Failing after 4s
/ check-panel (pull_request) Successful in 1m23s
/ check-deployment-basic (pull_request) Failing after 3s
/ check-deployment-cli (pull_request) Failing after 4s
/ check-deployment-panel (pull_request) Failing after 5s
to 3f63ca75d1
All checks were successful
/ check-pre-commit (pull_request) Successful in 12s
/ check-data-model (pull_request) Successful in 19s
/ check-peertube (pull_request) Successful in 21s
/ check-panel (pull_request) Successful in 1m21s
/ check-deployment-basic (pull_request) Successful in 11m21s
/ check-deployment-cli (pull_request) Successful in 38m16s
/ check-deployment-panel (pull_request) Successful in 41m40s
2025-06-21 09:13:55 +02:00
Compare
kiara force-pushed declare-classes from 3f63ca75d1
All checks were successful
/ check-pre-commit (pull_request) Successful in 12s
/ check-data-model (pull_request) Successful in 19s
/ check-peertube (pull_request) Successful in 21s
/ check-panel (pull_request) Successful in 1m21s
/ check-deployment-basic (pull_request) Successful in 11m21s
/ check-deployment-cli (pull_request) Successful in 38m16s
/ check-deployment-panel (pull_request) Successful in 41m40s
to b53cdd761a
All checks were successful
/ check-pre-commit (pull_request) Successful in 13s
/ check-data-model (pull_request) Successful in 17s
/ check-peertube (pull_request) Successful in 20s
/ check-panel (pull_request) Successful in 1m23s
/ check-deployment-basic (pull_request) Successful in 11m37s
/ check-deployment-cli (pull_request) Successful in 39m3s
/ check-deployment-panel (pull_request) Successful in 42m9s
2025-06-22 20:04:55 +02:00
Compare
kiara force-pushed declare-classes from b53cdd761a
All checks were successful
/ check-pre-commit (pull_request) Successful in 13s
/ check-data-model (pull_request) Successful in 17s
/ check-peertube (pull_request) Successful in 20s
/ check-panel (pull_request) Successful in 1m23s
/ check-deployment-basic (pull_request) Successful in 11m37s
/ check-deployment-cli (pull_request) Successful in 39m3s
/ check-deployment-panel (pull_request) Successful in 42m9s
to 109ce6a513
All checks were successful
/ check-pre-commit (pull_request) Successful in 14s
/ check-data-model (pull_request) Successful in 20s
/ check-peertube (pull_request) Successful in 21s
/ check-panel (pull_request) Successful in 1m29s
/ check-deployment-basic (pull_request) Successful in 11m32s
/ check-deployment-cli (pull_request) Successful in 39m10s
/ check-deployment-panel (pull_request) Successful in 41m50s
2025-06-22 20:07:06 +02:00
Compare
kiara force-pushed declare-classes from 109ce6a513
All checks were successful
/ check-pre-commit (pull_request) Successful in 14s
/ check-data-model (pull_request) Successful in 20s
/ check-peertube (pull_request) Successful in 21s
/ check-panel (pull_request) Successful in 1m29s
/ check-deployment-basic (pull_request) Successful in 11m32s
/ check-deployment-cli (pull_request) Successful in 39m10s
/ check-deployment-panel (pull_request) Successful in 41m50s
to 40a25beb4d
Some checks failed
/ check-pre-commit (pull_request) Has been cancelled
/ check-data-model (pull_request) Has been cancelled
/ check-peertube (pull_request) Has been cancelled
/ check-panel (pull_request) Has been cancelled
/ check-deployment-basic (pull_request) Has been cancelled
/ check-deployment-cli (pull_request) Has been cancelled
/ check-deployment-panel (pull_request) Has been cancelled
2025-06-23 17:24:48 +02:00
Compare
kiara merged commit 9a25a04bfa into main 2025-06-23 17:24:55 +02:00
kiara deleted branch declare-classes 2025-06-23 17:24:56 +02:00
Sign in to join this conversation.
No reviewers
No milestone
No project
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
fediversity/fediversity!398
No description provided.