Implement min reviewer branch policy + build validation + fix docs links (#326)

* Implement Min Reviewer Branch Policy

This change also does the following:

- Implement core components that can be shared across other policies
- Fix some documentation issues

* Adding support for build validation policy

* making acc tests more consistent'

* Update azuredevops/crud/branchpolicy/crud_branch_policy.go

Co-authored-by: xuzhang3 <57888764+xuzhang3@users.noreply.github.com>

* Update azuredevops/resource_branchpolicy_acceptance_test.go

Co-authored-by: xuzhang3 <57888764+xuzhang3@users.noreply.github.com>

* formatting fix

* docs fix

Co-authored-by: xuzhang3 <57888764+xuzhang3@users.noreply.github.com>

1 files changed, 2 insertions, 2 deletions

diff --git a/scripts/fmt-check-go.sh b/scripts/fmt-check-go.sh
index 9f25adfa..bf54b9f4 100755
--- a/scripts/fmt-check-go.sh
+++ b/scripts/fmt-check-go.sh
@@ -4,14 +4,14 @@ set -euo pipefail
 
 . $(dirname $0)/commons.sh
 
-info "Formatting Go Files... If this fails, run 'find . -name \"*.go\" | xargs gofmt -s -w' to fix"
+info "Formatting Go Files... If this fails, run 'find . -name \"*.go\" | grep -v 'vendor' | xargs gofmt -s -w' to fix"
 (
     cd "$SOURCE_DIR"
 
     # This runs a go fmt on each file without using the 'go fmt ./...' syntax.
     # This is advantageous because it avoids having to download all of the go
     # dependencies that would have been triggered by using the './...' syntax.
-    FILES_WITH_FMT_ISSUES=$(find . -name "*.go" | grep -v 'vendor'|  xargs gofmt -s -l | wc -l)
+    FILES_WITH_FMT_ISSUES=$(find . -name "*.go" | grep -v 'vendor' |  xargs gofmt -s -l | wc -l)
 
     # convert to integer...
     FILES_WITH_FMT_ISSUES=$(($FILES_WITH_FMT_ISSUES + 0))