nus-cs2113-AY2223S1 / forum

4 stars 0 forks source link

Configuration for protectecting the master branch in PR #20

Closed kaseykwok closed 1 year ago

kaseykwok commented 1 year ago

Can I know if this is the correct way to protect the PR from being merged in the master branch? I saw that there are no steps in the course website that told us to specify the required rules. The PR is allowed to merge despite failing the CI. So I fixed the configuration to this. Is it needed to configure it that way?

image
okkhoy commented 1 year ago

We don't mandate setting up branch protection rules. You can set it up by specifying the rules that you see fit. More details on the rules and how to go about it: