Remove vendor directory (#53)

* Remove the vendor directory

* Add the vendor directory to .gitignore

* Adapt Makefile for vendorless tests

* Remove module vendoring on test workflow

Also:
  * Simplify workflow triggers
  * Bump Go to 1.16 for consistency with the release workflow

Author: 0x2b3bfa0

.github/workflows/test.yml

@@ -1,8 +1,6 @@
 on:
   push:
-    branches: "*"
   pull_request:
-    branches: "*"
   workflow_dispatch:
 
 jobs:
@@ -14,14 +12,11 @@ jobs:
     - name: Install
       uses: actions/setup-go@v2
       with:
-        go-version: ^1.14
+        go-version: ^1.16
 
     - name: Checkout
       uses: actions/checkout@v2
 
-    - name: Vendor # FIXME: remove this step when migrating to pure Go modules.
-      run: go mod vendor
-
     - name: Build
       run: make build
 

.gitignore

@@ -36,4 +36,7 @@ override.tf.json
 # Ignore CLI configuration files
 .terraformrc
 terraform.rc
-terraform
+terraform
+
+# Ignore the Go vendor directory, superseded by proxy.golang.org and go.{mod,sum}
+vendor

Makefile

@@ -1,4 +1,3 @@
-TEST?=$$(go list ./... | grep -v 'vendor')
 HOSTNAME=github.com
 NAMESPACE=iterative
 NAME=iterative
@@ -16,9 +15,8 @@ install: build
 	mkdir -p ~/.terraform.d/plugins/${HOSTNAME}/${NAMESPACE}/${NAME}/${VERSION}/${OS_ARCH}
 	mv ${BINARY} ~/.terraform.d/plugins/${HOSTNAME}/${NAMESPACE}/${NAME}/${VERSION}/${OS_ARCH}
 
-test: 
-	go test -i $(TEST) || exit 1                                                   
-	echo $(TEST) | xargs -t -n4 go test $(TESTARGS) -timeout=30s -parallel=4                    
+test:
+	go test ./... $(TESTARGS) -timeout=30s -parallel=4
 
-testacc: 
-	TF_ACC=1 go test $(TEST) -v $(TESTARGS) -timeout 120m   
+testacc:
+	TF_ACC=1 go test ./... -v $(TESTARGS) -timeout 120m