search

kneasle / wheatley

An AI for Ringing Room that can ring any number of bells to increase the scope of practices.
https://pypi.org/project/wheatley/
MIT License
15 stars 13 forks source link

Make `/rebase` re-run tests #189

Closed kneasle closed 3 years ago

kneasle commented 3 years ago

Apparently we need a personal access token... This would make a good first issue, but unfortunately requires access to the security settings of the repo, which can only be done by me (@kneasle).

kneasle commented 3 years ago

Update: I suspect that this might not be possible because GitHub is trying to stop people crypto-mining with GH actions, and one side effect of this is that triggering CI runs is made harder.

centreboard commented 3 years ago

This makes the action useless as we can't even manually kick of checks after it pushes: https://github.com/kneasle/wheatley/pull/185/checks

kneasle commented 3 years ago

Yeah it does... tbh I'm sort of tempted to remove the /rebase command anyway, cos I've come to the conclusion that semi-linear commit histories are just way more effort than they're worth. And also the /rebase action gets run on every PR comment which clutters our CI history.

centreboard commented 3 years ago

:+1: Squash small changes and manually rebase then merge when useful to see separate commits?