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

feat: add +nightly option for rustfmt #174

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

ifiokjr
Copy link

@ifiokjr ifiokjr commented Mar 19, 2025

Description

This update introduces a new command-line argument --rustfmt-args to enable rustfmt with additional args like +nightly when using leptosfmt. The run_rustfmt function has been modified to accept this new argument, allowing for more flexible formatting options.

Currently no tests have been added. I can do so but it would require additional code changes to allow for ensuring the environment supports nightly rustfmt. Let me know if this would be desired.

Fixes #173

This update introduces a new command-line argument `--nightly` to enable formatting with rustfmt nightly after using leptosfmt. The `run_rustfmt` function has been modified to accept this new argument, allowing for more flexible formatting options.
@bram209
Copy link
Owner

bram209 commented Mar 20, 2025

Hi, thanks for the PR. I think I would rather have an “rustfmt extra args” argument similar to what RA has: rust-analyzer.rustfmt.extraArgs

Thoughts?

@ifiokjr
Copy link
Author

ifiokjr commented Mar 20, 2025

Yeah that make sense. I'll push an update.

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.

Add support for rustfmt +nightly
2 participants