exer_log icon indicating copy to clipboard operation
exer_log copied to clipboard

Setup `mergeable` to enforce coding standards

Open tenshiAMD opened this issue 2 years ago • 7 comments

I would like to suggest using mergeable to enforce coding standards. In my experience, this will help in maintaining the codebase.

tenshiAMD avatar Aug 20 '22 17:08 tenshiAMD

@KalleHallden Firstly, we need to follow this installation guide to make this work. Thank you.

tenshiAMD avatar Aug 20 '22 17:08 tenshiAMD

@KalleHallden also we need to declare who are the main reviewers so we can enable the auto-merge feature if you want,

tenshiAMD avatar Aug 21 '22 14:08 tenshiAMD

@KalleHallden hi now that we have other maintainers, we can add them too as required reviewers for a PR to be merged (on your behalf), what do you think?

tenshiAMD avatar Aug 24 '22 15:08 tenshiAMD

@KalleHallden hi now that we have other maintainers, we can add them too as required reviewers for a PR to be merged (on your behalf), what do you think?

Please do that. I was wondering about this. In total we have three maintainers.

  1. @jorre127
  2. @tenshiAMD
  3. @dinurymomshad

momshaddinury avatar Aug 24 '22 16:08 momshaddinury

@dinurymomshad yes that's what I mean

@KalleHallden also please add branch protection rules for the main branch (alpha) and master like this and check all mergeable status checks, thanks in advance

branch protection rules

tenshiAMD avatar Aug 24 '22 19:08 tenshiAMD

@KalleHallden also please add branch protection rules for the main branch (alpha) and master like this and check all mergeable status checks, thanks in advance

@KalleHallden hi may I follow this one? Thanks!

tenshiAMD avatar Sep 02 '22 19:09 tenshiAMD

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

stale[bot] avatar Sep 21 '22 12:09 stale[bot]