-
Notifications
You must be signed in to change notification settings - Fork 66
Closed
Labels
questionFurther information is requestedFurther information is requested
Description
Hi @AllenDang
thank you for inviting me to collaborating on this repository!
I've a proposal about setting up a branch-protection-rules.
In my opinion the protection rule should now enforce at least one PR review. This way, one person would give approval, and another one will merge. This will reduce a possibility of merging bugged code to master 😄
we should also add some auto-tests. e.g. a simple workflow file that will run go test ./... to ensure the code isn't buildable
Metadata
Metadata
Assignees
Labels
questionFurther information is requestedFurther information is requested