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

[pre-commit.ci] pre-commit autoupdate #270

Closed
wants to merge 2 commits into from

Conversation

pre-commit-ci[bot]
Copy link
Contributor

@pre-commit-ci pre-commit-ci bot commented Jan 29, 2024

@coveralls
Copy link
Collaborator

coveralls commented Jan 29, 2024

Pull Request Test Coverage Report for Build 7699666217

  • 0 of 2 (100.0%) changed or added relevant lines in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 99.779%

Totals Coverage Status
Change from base Build 7648049836: 0.0%
Covered Lines: 901
Relevant Lines: 903

💛 - Coveralls

@rmanaem rmanaem added the _bot [BOT only] Issue or PR made by a bot. label Feb 5, 2024
@surchs surchs self-requested a review February 12, 2024 15:45
@surchs
Copy link
Contributor

surchs commented Feb 12, 2024

Aha, I guess this guy is not just dealing with the pre-commit dependencies themselves? I believe @alyssadai you talked about this one, right?

@alyssadai
Copy link
Contributor

Yes! Did we agree to merge this, and then run black again to see if pre-commit makes further changes?

@surchs
Copy link
Contributor

surchs commented Feb 13, 2024

We're going to close this PR, see if black needs running locally, and then let pre-commit give this another try without touching the actual codebase.

@surchs surchs closed this Feb 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
_bot [BOT only] Issue or PR made by a bot.
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

4 participants