Skip to content

Remove --line-length option from format command - #8131

Merged
MichaReiser merged 1 commit into
mainfrom
remove-line-length-from-format
Oct 23, 2023
Merged

Remove --line-length option from format command#8131
MichaReiser merged 1 commit into
mainfrom
remove-line-length-from-format

Conversation

@MichaReiser

@MichaReiser MichaReiser commented Oct 23, 2023

Copy link
Copy Markdown
Member

Summary

This PR removes the undocumented --line-length setting from the format command.
The main reason is that we should either support all formatter options or none of them. I would prefer not supporting formatter options via the CLI.

We can reintroduce the option in the beta based on user feedback.

Test Plan

cargo run --bin ruff -q -- format . --line-length=100
error: unexpected argument '--line-length' found

@MichaReiser

MichaReiser commented Oct 23, 2023

Copy link
Copy Markdown
Member Author

Current dependencies on/for this PR:

This comment was auto-generated by Graphite.

@MichaReiser MichaReiser added cli Related to the command-line interface of Ruff or ty formatter Related to the Ruff formatter labels Oct 23, 2023
@MichaReiser MichaReiser added this to the Formatter: Beta milestone Oct 23, 2023
@github-actions

github-actions Bot commented Oct 23, 2023

Copy link
Copy Markdown
Contributor

PR Check Results

Ecosystem

✅ ecosystem check detected no changes.

@MichaReiser
MichaReiser merged commit 2f32a57 into main Oct 23, 2023
@MichaReiser
MichaReiser deleted the remove-line-length-from-format branch October 23, 2023 23:58
@p-linnane p-linnane mentioned this pull request Oct 24, 2023
Merged
This was referenced Oct 27, 2023
Closed
Closed
zanieb added a commit that referenced this pull request Oct 30, 2023
@zanieb
Revert "Remove --line-length option from format command (#8131)"
7a4cd39 
This reverts commit 2f32a57.
@zanieb zanieb mentioned this pull request Oct 30, 2023
Merged
zanieb added a commit that referenced this pull request Nov 2, 2023
@zanieb
Add --line-length option to format command (#8363)
3a889f4 
Restores the `--line-length` option removed in
#8131

Closes #8362
Closes #8352
@pipme

pipme commented Feb 20, 2024

Copy link
Copy Markdown

So what is the way to set a global line length that works across projects in VS Code? astral-sh/ruff-vscode#6

@MichaReiser

MichaReiser commented Feb 20, 2024

Copy link
Copy Markdown
Member Author

You can use --config line-length=100 in the most recent version of ruff

@pipme

pipme commented Feb 20, 2024

Copy link
Copy Markdown

Em, it seems not working. Is the following correct? I am using ruff v0.2.2.

  "ruff.path": ["/opt/homebrew/bin/ruff"],
  "ruff.args": [
    "--config",
    "line-length=100"
  ],

@MichaReiser

MichaReiser commented Feb 20, 2024

Copy link
Copy Markdown
Member Author

The option should either be ruff.lint.args or ruff.format.args. See configuring ruff

@pipme

pipme commented Feb 20, 2024

Copy link
Copy Markdown

Thanks! "ruff.format.args": ["--config", "line-length=79"] works.

KotlinIsland pushed a commit to KotlinIsland/basedpython that referenced this pull request May 1, 2026
@zanieb
Add --line-length option to format command (#8363)
33e0070 
Restores the `--line-length` option removed in
astral-sh/ruff#8131

Closes #8362
Closes #8352
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@charliermarsh charliermarsh charliermarsh approved these changes
@zanieb zanieb zanieb approved these changes
@konstin konstin konstin approved these changes

Assignees

No one assigned

Labels

cli Related to the command-line interface of Ruff or ty formatter Related to the Ruff formatter

Projects

None yet

Milestone

Formatter: Beta

Development

Successfully merging this pull request may close these issues.

5 participants

@MichaReiser @pipme @charliermarsh @zanieb @konstin