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.

Sign up for GitHub

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

Jump to bottom

Indent wheels like dependencies in the lockfile #4582

Merged
merged 1 commit into from
Jun 27, 2024
Merged

Indent wheels like dependencies in the lockfile #4582

merged 1 commit into from
Jun 27, 2024

Conversation

konstin
Copy link
Member

@konstin konstin commented Jun 27, 2024 •
edited
Loading

This PR contains two style changes to the lockfile:

  • Always indent lists of objects, even with they are only a single element.
  • Use 4 spaces instead of tabs for indenting, to mirror what we do in the ruff formatter.

@konstin konstin added the preview Experimental behavior label Jun 27, 2024
@konstin konstin mentioned this pull request Jun 27, 2024
Merged
Base automatically changed from konsti/indent-dependencies to main June 27, 2024 18:06
@konstin konstin force-pushed the konsti/indent-wheels-like-dependencies branch from d9f6968 to 24a00a7 Compare June 27, 2024 18:07
@konstin konstin marked this pull request as ready for review June 27, 2024 18:07
@konstin konstin changed the title Indent wheels like dependencies Indent wheels like dependencies in the lockfile Jun 27, 2024
@konstin konstin merged commit 80e45d3 into main Jun 27, 2024
47 checks passed
@konstin konstin deleted the konsti/indent-wheels-like-dependencies branch June 27, 2024 20:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@BurntSushi BurntSushi BurntSushi approved these changes

@ibraheemdev ibraheemdev ibraheemdev approved these changes

Assignees
No one assigned
Labels
preview Experimental behavior
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@konstin @BurntSushi @ibraheemdev