Skip to content

Commit

Permalink
provider: Update Makefile fmt target to ignore ./awsproviderlint/vend…
Browse files Browse the repository at this point in the history
…or (#15426)

Reference: #15391

Previously was not an issue before because we were vendoring in the root directory and explicitly referencing package subdirectories. Now that a vendor directory is included in ./awsproviderlint/ we need to exclude it. Using Makefile's built-in filter-out and wildcard functions to avoid platform specific issues with find, grep, etc.
  • Loading branch information
bflad authored Oct 1, 2020
1 parent fd656dd commit 54468a7
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion GNUmakefile
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,7 @@ testacc: fmtcheck

fmt:
@echo "==> Fixing source code with gofmt..."
gofmt -s -w ./$(PKG_NAME) ./awsproviderlint
gofmt -s -w ./$(PKG_NAME) $(filter-out ./awsproviderlint/go% ./awsproviderlint/README.md ./awsproviderlint/vendor, $(wildcard ./awsproviderlint/*))

# Currently required by tf-deploy compile
fmtcheck:
Expand Down

0 comments on commit 54468a7

Please sign in to comment.