Open Tohaker opened 12 months ago
I totally agree with all these points.
Till now it's just been myself and so now it needs these listed to make it easier for everyone to contribute.
I have added the necessary protection to the main branch and so now PRs require those checks.
Plus squash is now the only allowed option.
Added a new requirement - auto deleting branches after they've been merged. Helps keep the branches clean
Since you are already using Github Actions, consider using Changests with its automated Github Action.
With multiple developers contributing, we should have some rules in place to ensure the
main
branch stays clean and tidy. I have some suggestions based on previous experience, very much open to discussion though;Quality
jobs to pass in CI before a PR can be mergedSquash and merge
as the only merge optionCONTRIBUTING.md
guidelines - there's plenty of examples in the community for this