Skip to content

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

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

Closed
wants to merge 1 commit into from

Conversation

pre-commit-ci[bot]
Copy link
Contributor

@pre-commit-ci pre-commit-ci bot commented Dec 7, 2020

No description provided.

@MarcoGorelli
Copy link
Contributor

MarcoGorelli commented Dec 7, 2020

@twiecki looks like it's still installed - as the consensus among the rest of the devs is to remove it, do you want to uninstall it? I don't think I have permission for that

I can set up a GH action to do this 'autoupdate' weekly without pre-commit-ci

@codecov
Copy link

codecov bot commented Dec 7, 2020

Codecov Report

Merging #4312 (50235a7) into master (a9806db) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #4312   +/-   ##
=======================================
  Coverage   87.57%   87.57%           
=======================================
  Files          88       88           
  Lines       14274    14274           
=======================================
  Hits        12500    12500           
  Misses       1774     1774           

@MarcoGorelli
Copy link
Contributor

This action must be performed by an organization owner

yup, just tried and I can't remove it myself

@twiecki
Copy link
Member

twiecki commented Dec 8, 2020 via email

@twiecki twiecki closed this Dec 8, 2020
@twiecki twiecki deleted the pre-commit-ci-update-config branch December 8, 2020 08:41
@MarcoGorelli
Copy link
Contributor

@twiecki why delete this PR though? Updating the config file seemed like something we'd want

@twiecki
Copy link
Member

twiecki commented Dec 8, 2020

Oh, I didn't look, but the branch is gone now.

@MarcoGorelli
Copy link
Contributor

no worries 😄 I'll do that as part some other PR I need to open anyway

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.

2 participants