Skip to content

Conversation

egithinji
Copy link
Member

As mentioned in CONTRIBUTING.md, dprint is used for formatting in the project. This PR adds dprint to the list of packages that are installed when one runs cargo xtask install-tools.

@djmitche
Copy link
Collaborator

I suspect the build failures were some sort of intermittent? I can't merge without them passing, but if necessary someone with admin privileges can.

@egithinji
Copy link
Member Author

I suspect the build failures were some sort of intermittent? I can't merge without them passing, but if necessary someone with admin privileges can.

Not sure but might be related to rust-lang/rust#143835. So as not to break the CI might be better if it's resolved first. I'll keep an eye on it and dig deeper when I get a chance.

@gribozavr gribozavr added the waiting for author Waiting on the issue author to reply label Aug 26, 2025
@egithinji
Copy link
Member Author

Once dprint/dprint#1021 lands, we can upgrade our dprint version and I believe this should fix the build issues.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
waiting for author Waiting on the issue author to reply
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants