New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

updated branchtest to skip approval for contibutors and authors #3146

Merged

abhishek9686 merged 2 commits into master from NET-1632-master

Oct 2, 2024

Collaborator

lmnzx commented Oct 1, 2024

Describe your changes

Provide Issue ticket number if applicable/not in title

Provide testing steps

Checklist before requesting a review

My changes affect only 10 files or less.
I have performed a self-review of my code and tested it.
If it is a new feature, I have added thorough tests, my code is <= 1450 lines.
If it is a bugfix, my code is <= 200 lines.
My functions are <= 80 lines.
I have had my code reviewed by a peer.
My unit tests pass locally.
Netmaker is awesome.


          updated branchtest to skip approval for contibutors and authors

62885aa

abhishek9686 approved these changes

View reviewed changes


          removed retriggering of workflow from approval for internal prs

26763ac

abhishek9686 approved these changes

View reviewed changes

abhishek9686 merged commit d5f885d into master

6 checks passed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet