qo.is/infrastructure
3
0
Fork 0
Code Issues 10 Pull requests Projects Activity Actions

Many small structural improvements (treefmt, no cyclic imports, git-pre-commit-hook, faster CI) #50

Merged
fabianhauser merged 16 commits from update-imports into main 2025-03-26 15:10:42 +01:00
Conversation 0 Commits 16 Files changed 106 +864 -780
fabianhauser commented 2025-03-25 13:37:11 +01:00
Owner
Copy link
  • Add treefmt (-nix) to format all files.
  • Add git-pre-commit hook to run treefmt
  • Resolve the cyclic imports that we had. This is important for writing more tests in the future.
  • Simplify and speed up CI build
    • Run all builds with nix flake check
    • Use nix-fast-build in CI for parallel builds and nix develop environment.
  • Clean up substitutors configuration.
  • Fix some small bugs alighted brought to light by above refactorings
- Add `treefmt` (`-nix`) to format all files. - Add git-pre-commit hook to run `treefmt` - Resolve the cyclic imports that we had. This is important for writing more tests in the future. - Simplify and speed up CI build - Run all builds with `nix flake check` - Use `nix-fast-build` in CI for parallel builds and `nix develop` environment. - Clean up substitutors configuration. - Fix some small bugs alighted brought to light by above refactorings
fabianhauser self-assigned this 2025-03-25 13:37:11 +01:00
fabianhauser added 4 commits 2025-03-25 13:37:12 +01:00
Add pre-commit-hook with formatting
Some checks failed
CI / build (push) Failing after 47s
Details
cad4f35ee9
fabianhauser added 1 commit 2025-03-25 14:05:08 +01:00
Update build process ci and docs
Some checks failed
CI / build (push) Failing after 15s
Details
172e4c37a3
fabianhauser added 1 commit 2025-03-25 14:12:58 +01:00
Update devshell to include sed
Some checks failed
CI / build (push) Failing after 15s
Details
f3e2c00331
fabianhauser force-pushed update-imports from f3e2c00331 to 172e4c37a3 2025-03-25 14:20:26 +01:00 Compare
fabianhauser added 1 commit 2025-03-25 14:23:47 +01:00
Extend package set available in git-ci-runner
All checks were successful
CI / build (push) Successful in 1m49s
Details
fc39e912ad
fabianhauser added 8 commits 2025-03-25 22:59:57 +01:00
Disable backup and outgoing mail in module test
Some checks failed
CI / build (push) Has been cancelled
Details
afd284a008
fabianhauser changed title from Update flake imports to prefent infinite recursions and add treefmt & git-hooks for formatting to Many small structural improvements (treefmt, no cyclic imports, faster CI) 2025-03-25 23:01:04 +01:00
fabianhauser changed title from Many small structural improvements (treefmt, no cyclic imports, faster CI) to Many small structural improvements (treefmt, no cyclic imports, git-pre-commit-hook, faster CI) 2025-03-25 23:01:42 +01:00
fabianhauser added 2 commits 2025-03-26 15:00:13 +01:00
Add note about clearing git-ci-runner caches
All checks were successful
CI / build (push) Successful in 2m13s
Details
c9cde808e2
fabianhauser merged commit c9cde808e2 into main 2025-03-26 15:10:42 +01:00
fabianhauser deleted branch update-imports 2025-03-26 15:10:42 +01:00
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
bug

Something is not working

  
duplicate

This issue or pull request already exists

  
enhancement

New feature

  
help wanted

Need some help

  
invalid

Something is wrong

  
question

More information is needed

  
urgent

This issue should be tackled soon

  
wontfix

This won't be fixed

No labels
bug
duplicate
enhancement
help wanted
invalid
question
urgent
wontfix
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
fabianhauser
1 participant
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: qo.is/infrastructure#50
No description provided.
Delete branch "update-imports"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?