dweb-mirror icon indicating copy to clipboard operation
dweb-mirror copied to clipboard

Consider Automated Pull Request Merging,Optionally After Sufficient Review

Open xloem opened this issue 4 years ago • 2 comments

See https://docs.github.com/en/github/collaborating-with-issues-and-pull-requests/automatically-merging-a-pull-request .

This project looks pretty important, so it would be great to make sure there are ways for it to continue on its own when the maintainers are busy.

xloem avatar Jan 20 '21 12:01 xloem

Hi, I'm not sure auto-merge is going to work, partly because currently I'm the only one who has spent a lot of time on the code, so I might catch something otherwise missed, and if I have to review anyway then its just as easy for me to click the "merge" button at the same time.

Even though I'm no longer working on the project, I'm generally pretty available for code-review and merges, though I'm on Australia time.

Glad you are working on it - thanks for catching problems with the installer - which is a bit surprising given that I've had it working on multiple platforms (OSX, Intel, OLIP, IIAB, Rachel), the OSX is pretty cooked - i.e. I've already got stuff installed - but the other platforms are raw out-of-the-box installs.

mitra42 avatar Jan 21 '21 01:01 mitra42

By the way ... if you are testing and in a position to confirm that yarn update or similar gets the newer packages with no problems I could merge the dependabot PR's -

mitra42 avatar Jan 21 '21 01:01 mitra42