Skip to content

fix: For treefmt in CI, use --no-cache#44

Merged
RasmusRendal merged 1 commit into
mainfrom
RasmusRendal/treefmt_no_cache
Jul 21, 2026
Merged

fix: For treefmt in CI, use --no-cache#44
RasmusRendal merged 1 commit into
mainfrom
RasmusRendal/treefmt_no_cache

Conversation

@RasmusRendal

Copy link
Copy Markdown
Contributor

It was breaking some downstream users of ES, and is recommended by the
official docs.

@RasmusRendal
RasmusRendal force-pushed the RasmusRendal/treefmt_no_cache branch from 1fcd21c to f2cc2a0 Compare July 21, 2026 08:18
It was breaking some downstream users of ES, and is recommended by the
official docs.
@RasmusRendal
RasmusRendal force-pushed the RasmusRendal/treefmt_no_cache branch from f2cc2a0 to 39f7852 Compare July 21, 2026 08:22
@RasmusRendal
RasmusRendal marked this pull request as ready for review July 21, 2026 08:35
@cursor

cursor Bot commented Jul 21, 2026

Copy link
Copy Markdown

PR Summary

Low Risk
CI-only environment variable for the standards pre-commit job; no runtime or application logic changes.

Overview
Sets TREEFMT_NO_CACHE=1 on the CI step that runs prek --all-files so treefmt skips its cache during pre-commit checks.

The change is defined in check-pre-commit-hooks.nix (the workflow YAML is generated). Comments note this avoids cache timeouts on some runners and aligns with treefmt’s CI guidance; --fail-on-change is not added because prek already enforces formatting failures.

Reviewed by Cursor Bugbot for commit 39f7852. Bugbot is set up for automated code reviews on this repo. Configure here.

@RasmusRendal
RasmusRendal merged commit 39f7852 into main Jul 21, 2026
4 checks passed
@RasmusRendal
RasmusRendal deleted the RasmusRendal/treefmt_no_cache branch July 21, 2026 08:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants