Skip to content

Conversation

@vedantgrover
Copy link
Collaborator

No description provided.

@Williamwu277
Copy link
Contributor

damn thats a lot of stuff

@vedantgrover
Copy link
Collaborator Author

damn thats a lot of stuff

lol yeah a lot of it is the pre-commit fixing anything that violates its terms.

@vedantgrover
Copy link
Collaborator Author

Okay last commit should've fixed everything (while keeping everything working).

Williamwu277
Williamwu277 previously approved these changes Oct 30, 2025
Copy link
Contributor

@Williamwu277 Williamwu277 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should be good to go. Ran on my branch

Copy link
Contributor

@Williamwu277 Williamwu277 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!!!

@vedantgrover vedantgrover merged commit ff43a43 into master Nov 7, 2025
1 check passed
@vedantgrover vedantgrover deleted the add-pre-commit branch November 7, 2025 23:14
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.

4 participants