Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

numtide/treefmt compatibility #88

Closed
yorickvP opened this issue Feb 25, 2022 · 2 comments · Fixed by #133
Closed

numtide/treefmt compatibility #88

yorickvP opened this issue Feb 25, 2022 · 2 comments · Fixed by #133

Comments

@yorickvP
Copy link
Contributor

User report:

I set up treefmt for my editor and nixfmt fails formatting without any obvious sign.
Maybe the spec is incomplete, but I think it had to do with some sort of temp-file mutation. iirc.
Maybe treefmt and nixfmt are competing on the file-handling domain in incompatible ways?

@yorickvP
Copy link
Contributor Author

@blaggacao

@SebTM
Copy link

SebTM commented Sep 24, 2023

I would like to hijack this as nixfmt was added by me to treefmt without seeing this - it works fine (from formatting output) but it changes mtime even file didn't had to be reformatted which was reported here and violates the idempotence requirement from treefmt.

Sadly I don't have any Haskell knowledge and currently no time to get into so I would appreciate if someone could help out here ✌️

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 a pull request may close this issue.

2 participants