-
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
Closed
Labels
AutomationGithub bots, testing workflows, release automationGithub bots, testing workflows, release automation
Description
Currently we have two bots which comment on issues, I'd propose we remove them both
- The "Unit Test Results". I don't find this that useful — do others? I thought it would be listing the tests that failed (I had thought it might even be doing duration changes etc), but it seems to just list the number of tests that failed, which isn't that useful. Or are we not using it correctly?
- pep8speaks — I think this would be dominated by pre-commit.ci
I'd propose we add:
- pre-commit.ci. This autofixes PRs! e.g. here's a test PR with a black error from pytest-accept: Test commit for ci-pre-commit max-sixty/pytest-accept#24. This could also replace the pre-commit GHA (though it's no great cost to duplicate it)
Metadata
Metadata
Assignees
Labels
AutomationGithub bots, testing workflows, release automationGithub bots, testing workflows, release automation