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

Use the LinterSettings's tab size when expanding indent #9506

Merged
merged 2 commits into from Jan 14, 2024

Conversation

hoel-bagard
Copy link
Contributor

Summary

In the logical_lines's expand_indent , respect the LinterSettings::tab_size setting instead of hardcoding the size of tabs to 8.

Also see this conversation

Test Plan

Tested by running cargo test

Copy link
Contributor

github-actions bot commented Jan 13, 2024

ruff-ecosystem results

Linter (stable)

✅ ecosystem check detected no linter changes.

Linter (preview)

✅ ecosystem check detected no linter changes.

@hoel-bagard hoel-bagard changed the title Use the LinterSettings's tab size. Use the LinterSettings's tab size when expanding indent Jan 13, 2024
@zanieb zanieb added the linter Related to the linter label Jan 13, 2024
@charliermarsh charliermarsh merged commit e8d7a6d into astral-sh:main Jan 14, 2024
17 checks passed
@charliermarsh
Copy link
Member

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
linter Related to the linter
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants