Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

style: format with Ruff #65

Merged
merged 2 commits into from
Jun 26, 2024
Merged

style: format with Ruff #65

merged 2 commits into from
Jun 26, 2024

Conversation

jsstevenson
Copy link
Contributor

@jsstevenson jsstevenson commented Jun 5, 2024

Per a previous discussion, this PR implements only the formatting half of ruff.

@jsstevenson jsstevenson marked this pull request as ready for review June 18, 2024 11:55
@jsstevenson jsstevenson requested a review from a team as a code owner June 18, 2024 11:55
Copy link

@korikuzma korikuzma left a comment

Choose a reason for hiding this comment

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

@jsstevenson do you plan on adding a .git-blame-ignore-revs file? More here

pyproject.toml Outdated Show resolved Hide resolved
@jsstevenson
Copy link
Contributor Author

@korikuzma sure, the # of changes is fairly minimal but we can

@jsstevenson jsstevenson requested a review from korikuzma June 21, 2024 12:12
@korikuzma
Copy link

@korikuzma sure, the # of changes is fairly minimal but we can

@jsstevenson Ya, I think it'd be good to do for all the other ruff format PRs too

Copy link

@korikuzma korikuzma left a comment

Choose a reason for hiding this comment

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

See thread above (trying to remove from my requested reviews)

@jsstevenson
Copy link
Contributor Author

ope didn't commit

@jsstevenson jsstevenson requested a review from korikuzma June 26, 2024 12:18
@jsstevenson jsstevenson merged commit f82e37d into main Jun 26, 2024
13 checks passed
@jsstevenson jsstevenson deleted the ruff-format branch June 26, 2024 12:44
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants