Replies: 7 comments 2 replies
This comment has been hidden.
This comment has been hidden.
-
Totally agree |
Beta Was this translation helpful? Give feedback.
-
ops, sorry, wrong issue 😄
great!, Sadly, I can't do that, since I don't have access to settings, but the setup should look like that: |
Beta Was this translation helpful? Give feedback.
-
Another proposal: let's enable "Discussions" in this repo and move topics like these there instead of making many "issues" as they're usually for bugs/feature requests, etc. |
Beta Was this translation helpful? Give feedback.
-
it is done now. Converting this to discussion |
Beta Was this translation helpful? Give feedback.
-
@gucio321 I just created the branch protection rules |
Beta Was this translation helpful? Give feedback.
-
Hi! |
Beta Was this translation helpful? Give feedback.
-
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 buildableBeta Was this translation helpful? Give feedback.
All reactions