Skip to content

ci: fix formatting check erroring out in an incorrect manner#163

Merged
zimward merged 1 commit into
Lassulus:mainfrom
lineargraph:push-msymwnrptxmw
May 15, 2026
Merged

ci: fix formatting check erroring out in an incorrect manner#163
zimward merged 1 commit into
Lassulus:mainfrom
lineargraph:push-msymwnrptxmw

Conversation

@lineargraph
Copy link
Copy Markdown
Contributor

previously this would fail with a non descript openTempFileWithDefaultPermissions: permission denied (Permission denied)

now the correct file gets identified in the error log

@lineargraph
Copy link
Copy Markdown
Contributor Author

quoting https://github.com/Lassulus/wrappers/actions/runs/25938713491/job/76250654379?pr=163


       Output paths:
         /nix/store/5rxrdqviy44x0z7dq0i4546mg84hwkza-formatting-check
       Last 5 log lines:
       > ERRO file has changed path=checks/formatting.nix prev_size=395 prev_mod_time="2026-05-15 20:03:59 +0000 UTC" current_size=388 current_mod_time="2026-05-15 20:04:00 +0000 UTC"
       > traversed 109 files
       > emitted 103 files for processing
       > formatted 103 files (1 changed) in 995ms
       > Error: unexpected changes detected, --fail-on-change is enabled
       For full logs, run:


previously this would fail with a non descript openTempFileWithDefaultPermissions: permission denied (Permission denied)

now the correct file gets identified in the error log
Copy link
Copy Markdown
Collaborator

@zimward zimward left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yup that makes the CI fail a lot clearer

@zimward zimward merged commit 6e26cef into Lassulus:main May 15, 2026
2 checks passed
@lineargraph lineargraph deleted the push-msymwnrptxmw branch May 15, 2026 20:20
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