gocodebox / lifterlms

LifterLMS, a WordPress LMS Solution: Easily create, sell, and protect engaging online courses.
https://lifterlms.com
GNU General Public License v3.0
174 stars 134 forks source link

Don't merge open PRs in dev by default. #2633

Closed ideadude closed 2 months ago

ideadude commented 2 months ago

We're not sure why we would want to merge open PRs in the dev branch AFTER pushing a release. This commit disables that by setting the default for the 'merge' option to ''.

This didn't seem to be an issue in the past because failing tests and branch protection rules would stop any PRs from merging anyway.

Checklist:

github-actions[bot] commented 2 months ago

Coverage report

St.:grey_question:
Category Percentage Covered / Total
🟑 Statements 77.18% 328/425
🟑 Branches 68.45% 141/206
🟒 Functions 82.93% 68/82
🟑 Lines 76.68% 319/416

Test suite run success

169 tests passing in 22 suites.

Report generated by πŸ§ͺjest coverage report action from ef21d6c4be2b034a85be68569b64b75a3c6437c3