forked from wbernest/mattermost-plugin-rssfeed
-
Notifications
You must be signed in to change notification settings - Fork 0
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Make every file LF terminated to utilize gofmt
cf. - golang/go#16355 - golang/go#2242 Signed-off-by: Takuya Noguchi <takninnovationresearch@gmail.com>
- Loading branch information
Showing
5 changed files
with
738 additions
and
738 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,184 +1,184 @@ | ||
| GO ?= $(shell command -v go 2> /dev/null) | ||
| DEP ?= $(shell command -v dep 2> /dev/null) | ||
| NPM ?= $(shell command -v npm 2> /dev/null) | ||
| HTTP ?= $(shell command -v http 2> /dev/null) | ||
| CURL ?= $(shell command -v curl 2> /dev/null) | ||
| MANIFEST_FILE ?= plugin.json | ||
|
|
||
| # You can include assets this directory into the bundle. This can be e.g. used to include profile pictures. | ||
| ASSETS_DIR ?= assets | ||
|
|
||
| # Verify environment, and define PLUGIN_ID, PLUGIN_VERSION, HAS_SERVER and HAS_WEBAPP as needed. | ||
| include build/setup.mk | ||
|
|
||
| BUNDLE_NAME ?= $(PLUGIN_ID)-$(PLUGIN_VERSION).tar.gz | ||
|
|
||
| ## Checks the code style, tests, builds and bundles the plugin. | ||
| all: check-style test dist | ||
|
|
||
| ## Propagates plugin manifest information into the server/ and webapp/ folders as required. | ||
| .PHONY: apply | ||
| apply: | ||
| ./build/bin/manifest apply | ||
|
|
||
| ## Runs govet and gofmt against all packages. | ||
| .PHONY: check-style | ||
| check-style: server/.depensure webapp/.npminstall gofmt govet | ||
| @echo Checking for style guide compliance | ||
|
|
||
| ifneq ($(HAS_WEBAPP),) | ||
| cd webapp && npm run lint | ||
| endif | ||
|
|
||
| ## Runs gofmt against all packages. | ||
| .PHONY: gofmt | ||
| gofmt: | ||
| ifneq ($(HAS_SERVER),) | ||
| @echo Running gofmt | ||
| @for package in $$(go list ./server/...); do \ | ||
| echo "Checking "$$package; \ | ||
| files=$$(go list -f '{{range .GoFiles}}{{$$.Dir}}/{{.}} {{end}}' $$package); \ | ||
| if [ "$$files" ]; then \ | ||
| gofmt_output=$$(gofmt -d -s $$files 2>&1); \ | ||
| if [ "$$gofmt_output" ]; then \ | ||
| echo "$$gofmt_output"; \ | ||
| echo "Gofmt failure"; \ | ||
| exit 1; \ | ||
| fi; \ | ||
| fi; \ | ||
| done | ||
| @echo Gofmt success | ||
| endif | ||
|
|
||
| ## Runs govet against all packages. | ||
| .PHONY: govet | ||
| govet: | ||
| ifneq ($(HAS_SERVER),) | ||
| @echo Running govet | ||
| $(GO) get golang.org/x/tools/go/analysis/passes/shadow/cmd/shadow | ||
| $(GO) vet $$(go list ./server/...) | ||
| $(GO) vet -vettool=$(GOPATH)/bin/shadow $$(go list ./server/...) | ||
| @echo Govet success | ||
| endif | ||
|
|
||
| ## Ensures the server dependencies are installed. | ||
| server/.depensure: | ||
| ifneq ($(HAS_SERVER),) | ||
| cd server && $(DEP) ensure | ||
| touch $@ | ||
| endif | ||
|
|
||
| ## Builds the server, if it exists, including support for multiple architectures. | ||
| .PHONY: server | ||
| server: server/.depensure | ||
| ifneq ($(HAS_SERVER),) | ||
| mkdir -p server/dist; | ||
| cd server && env GOOS=linux GOARCH=amd64 $(GO) build -o dist/plugin-linux-amd64; | ||
| cd server && env GOOS=darwin GOARCH=amd64 $(GO) build -o dist/plugin-darwin-amd64; | ||
| cd server && env GOOS=windows GOARCH=amd64 $(GO) build -o dist/plugin-windows-amd64.exe; | ||
| endif | ||
|
|
||
| ## Ensures NPM dependencies are installed without having to run this all the time. | ||
| webapp/.npminstall: | ||
| ifneq ($(HAS_WEBAPP),) | ||
| cd webapp && $(NPM) install | ||
| touch $@ | ||
| endif | ||
|
|
||
| ## Builds the webapp, if it exists. | ||
| .PHONY: webapp | ||
| webapp: webapp/.npminstall | ||
| ifneq ($(HAS_WEBAPP),) | ||
| cd webapp && $(NPM) run build; | ||
| endif | ||
|
|
||
| ## Generates a tar bundle of the plugin for install. | ||
| .PHONY: bundle | ||
| bundle: | ||
| rm -rf dist/ | ||
| mkdir -p dist/$(PLUGIN_ID) | ||
| cp $(MANIFEST_FILE) dist/$(PLUGIN_ID)/ | ||
| ifneq ($(wildcard $(ASSETS_DIR)/.),) | ||
| cp -r $(ASSETS_DIR) dist/$(PLUGIN_ID)/ | ||
| endif | ||
| ifneq ($(HAS_SERVER),) | ||
| mkdir -p dist/$(PLUGIN_ID)/server/dist; | ||
| cp -r server/dist/* dist/$(PLUGIN_ID)/server/dist/; | ||
| endif | ||
| ifneq ($(HAS_WEBAPP),) | ||
| mkdir -p dist/$(PLUGIN_ID)/webapp/dist; | ||
| cp -r webapp/dist/* dist/$(PLUGIN_ID)/webapp/dist/; | ||
| endif | ||
| cd dist && tar -cvzf $(BUNDLE_NAME) $(PLUGIN_ID) | ||
|
|
||
| @echo plugin built at: dist/$(BUNDLE_NAME) | ||
|
|
||
| ## Builds and bundles the plugin. | ||
| .PHONY: dist | ||
| dist: apply server webapp bundle | ||
|
|
||
| ## Installs the plugin to a (development) server. | ||
| .PHONY: deploy | ||
| deploy: dist | ||
| ## It uses the API if appropriate environment variables are defined, | ||
| ## or copying the files directly to a sibling mattermost-server directory. | ||
| ifneq ($(and $(MM_SERVICESETTINGS_SITEURL),$(MM_ADMIN_USERNAME),$(MM_ADMIN_PASSWORD),$(HTTP)),) | ||
| @echo "Installing plugin via API" | ||
| (TOKEN=`http --print h POST $(MM_SERVICESETTINGS_SITEURL)/api/v4/users/login login_id=$(MM_ADMIN_USERNAME) password=$(MM_ADMIN_PASSWORD) | grep Token | cut -f2 -d' '` && \ | ||
| http --print b GET $(MM_SERVICESETTINGS_SITEURL)/api/v4/users/me Authorization:"Bearer $$TOKEN" && \ | ||
| http --print b DELETE $(MM_SERVICESETTINGS_SITEURL)/api/v4/plugins/$(PLUGIN_ID) Authorization:"Bearer $$TOKEN" && \ | ||
| http --print b --check-status --form POST $(MM_SERVICESETTINGS_SITEURL)/api/v4/plugins plugin@dist/$(BUNDLE_NAME) Authorization:"Bearer $$TOKEN" && \ | ||
| http --print b POST $(MM_SERVICESETTINGS_SITEURL)/api/v4/plugins/$(PLUGIN_ID)/enable Authorization:"Bearer $$TOKEN" && \ | ||
| http --print b POST $(MM_SERVICESETTINGS_SITEURL)/api/v4/users/logout Authorization:"Bearer $$TOKEN" \ | ||
| ) | ||
| else ifneq ($(and $(MM_SERVICESETTINGS_SITEURL),$(MM_ADMIN_USERNAME),$(MM_ADMIN_PASSWORD),$(CURL)),) | ||
| @echo "Installing plugin via API" | ||
| $(eval TOKEN := $(shell curl -i -X POST $(MM_SERVICESETTINGS_SITEURL)/api/v4/users/login -d '{"login_id": "$(MM_ADMIN_USERNAME)", "password": "$(MM_ADMIN_PASSWORD)"}' | grep Token | cut -f2 -d' ' 2> /dev/null)) | ||
| @curl -s -H "Authorization: Bearer $(TOKEN)" -X DELETE $(MM_SERVICESETTINGS_SITEURL)/api/v4/plugins/$(PLUGIN_ID) > /dev/null | ||
| @curl -s -H "Authorization: Bearer $(TOKEN)" -X POST $(MM_SERVICESETTINGS_SITEURL)/api/v4/plugins -F "plugin=@dist/$(BUNDLE_NAME)" > /dev/null && \ | ||
| curl -s -H "Authorization: Bearer $(TOKEN)" -X POST $(MM_SERVICESETTINGS_SITEURL)/api/v4/plugins/$(PLUGIN_ID)/enable > /dev/null && \ | ||
| echo "OK." || echo "Sorry, something went wrong." | ||
| else ifneq ($(wildcard ../mattermost-server/.*),) | ||
| @echo "Installing plugin via filesystem. Server restart and manual plugin enabling required" | ||
| mkdir -p ../mattermost-server/plugins | ||
| tar -C ../mattermost-server/plugins -zxvf dist/$(BUNDLE_NAME) | ||
| else | ||
| @echo "No supported deployment method available. Install plugin manually." | ||
| endif | ||
|
|
||
| ## Runs any lints and unit tests defined for the server and webapp, if they exist. | ||
| .PHONY: test | ||
| test: server/.depensure webapp/.npminstall | ||
| ifneq ($(HAS_SERVER),) | ||
| cd server && $(GO) test -race -v ./... | ||
| endif | ||
| ifneq ($(HAS_WEBAPP),) | ||
| cd webapp && $(NPM) run fix; | ||
| endif | ||
|
|
||
| ## Creates a coverage report for the server code. | ||
| .PHONY: coverage | ||
| coverage: server/.depensure webapp/.npminstall | ||
| ifneq ($(HAS_SERVER),) | ||
| cd server && $(GO) test -race -coverprofile=coverage.txt ./... | ||
| @cd server && $(GO) tool cover -html=coverage.txt | ||
| endif | ||
|
|
||
| ## Clean removes all build artifacts. | ||
| .PHONY: clean | ||
| clean: | ||
| rm -fr dist/ | ||
| ifneq ($(HAS_SERVER),) | ||
| rm -fr server/dist | ||
| rm -fr server/.depensure | ||
| endif | ||
| ifneq ($(HAS_WEBAPP),) | ||
| rm -fr webapp/.npminstall | ||
| rm -fr webapp/dist | ||
| rm -fr webapp/node_modules | ||
| endif | ||
| rm -fr build/bin/ | ||
|
|
||
| # Help documentatin à la https://marmelab.com/blog/2016/02/29/auto-documented-makefile.html | ||
| help: | ||
| @cat Makefile | grep -v '\.PHONY' | grep -v '\help:' | grep -B1 -E '^[a-zA-Z_.-]+:.*' | sed -e "s/:.*//" | sed -e "s/^## //" | grep -v '\-\-' | sed '1!G;h;$$!d' | awk 'NR%2{printf "\033[36m%-30s\033[0m",$$0;next;}1' | sort | ||
| GO ?= $(shell command -v go 2> /dev/null) | ||
| DEP ?= $(shell command -v dep 2> /dev/null) | ||
| NPM ?= $(shell command -v npm 2> /dev/null) | ||
| HTTP ?= $(shell command -v http 2> /dev/null) | ||
| CURL ?= $(shell command -v curl 2> /dev/null) | ||
| MANIFEST_FILE ?= plugin.json | ||
|
|
||
| # You can include assets this directory into the bundle. This can be e.g. used to include profile pictures. | ||
| ASSETS_DIR ?= assets | ||
|
|
||
| # Verify environment, and define PLUGIN_ID, PLUGIN_VERSION, HAS_SERVER and HAS_WEBAPP as needed. | ||
| include build/setup.mk | ||
|
|
||
| BUNDLE_NAME ?= $(PLUGIN_ID)-$(PLUGIN_VERSION).tar.gz | ||
|
|
||
| ## Checks the code style, tests, builds and bundles the plugin. | ||
| all: check-style test dist | ||
|
|
||
| ## Propagates plugin manifest information into the server/ and webapp/ folders as required. | ||
| .PHONY: apply | ||
| apply: | ||
| ./build/bin/manifest apply | ||
|
|
||
| ## Runs govet and gofmt against all packages. | ||
| .PHONY: check-style | ||
| check-style: server/.depensure webapp/.npminstall gofmt govet | ||
| @echo Checking for style guide compliance | ||
|
|
||
| ifneq ($(HAS_WEBAPP),) | ||
| cd webapp && npm run lint | ||
| endif | ||
|
|
||
| ## Runs gofmt against all packages. | ||
| .PHONY: gofmt | ||
| gofmt: | ||
| ifneq ($(HAS_SERVER),) | ||
| @echo Running gofmt | ||
| @for package in $$(go list ./server/...); do \ | ||
| echo "Checking "$$package; \ | ||
| files=$$(go list -f '{{range .GoFiles}}{{$$.Dir}}/{{.}} {{end}}' $$package); \ | ||
| if [ "$$files" ]; then \ | ||
| gofmt_output=$$(gofmt -d -s $$files 2>&1); \ | ||
| if [ "$$gofmt_output" ]; then \ | ||
| echo "$$gofmt_output"; \ | ||
| echo "Gofmt failure"; \ | ||
| exit 1; \ | ||
| fi; \ | ||
| fi; \ | ||
| done | ||
| @echo Gofmt success | ||
| endif | ||
|
|
||
| ## Runs govet against all packages. | ||
| .PHONY: govet | ||
| govet: | ||
| ifneq ($(HAS_SERVER),) | ||
| @echo Running govet | ||
| $(GO) get golang.org/x/tools/go/analysis/passes/shadow/cmd/shadow | ||
| $(GO) vet $$(go list ./server/...) | ||
| $(GO) vet -vettool=$(GOPATH)/bin/shadow $$(go list ./server/...) | ||
| @echo Govet success | ||
| endif | ||
|
|
||
| ## Ensures the server dependencies are installed. | ||
| server/.depensure: | ||
| ifneq ($(HAS_SERVER),) | ||
| cd server && $(DEP) ensure | ||
| touch $@ | ||
| endif | ||
|
|
||
| ## Builds the server, if it exists, including support for multiple architectures. | ||
| .PHONY: server | ||
| server: server/.depensure | ||
| ifneq ($(HAS_SERVER),) | ||
| mkdir -p server/dist; | ||
| cd server && env GOOS=linux GOARCH=amd64 $(GO) build -o dist/plugin-linux-amd64; | ||
| cd server && env GOOS=darwin GOARCH=amd64 $(GO) build -o dist/plugin-darwin-amd64; | ||
| cd server && env GOOS=windows GOARCH=amd64 $(GO) build -o dist/plugin-windows-amd64.exe; | ||
| endif | ||
|
|
||
| ## Ensures NPM dependencies are installed without having to run this all the time. | ||
| webapp/.npminstall: | ||
| ifneq ($(HAS_WEBAPP),) | ||
| cd webapp && $(NPM) install | ||
| touch $@ | ||
| endif | ||
|
|
||
| ## Builds the webapp, if it exists. | ||
| .PHONY: webapp | ||
| webapp: webapp/.npminstall | ||
| ifneq ($(HAS_WEBAPP),) | ||
| cd webapp && $(NPM) run build; | ||
| endif | ||
|
|
||
| ## Generates a tar bundle of the plugin for install. | ||
| .PHONY: bundle | ||
| bundle: | ||
| rm -rf dist/ | ||
| mkdir -p dist/$(PLUGIN_ID) | ||
| cp $(MANIFEST_FILE) dist/$(PLUGIN_ID)/ | ||
| ifneq ($(wildcard $(ASSETS_DIR)/.),) | ||
| cp -r $(ASSETS_DIR) dist/$(PLUGIN_ID)/ | ||
| endif | ||
| ifneq ($(HAS_SERVER),) | ||
| mkdir -p dist/$(PLUGIN_ID)/server/dist; | ||
| cp -r server/dist/* dist/$(PLUGIN_ID)/server/dist/; | ||
| endif | ||
| ifneq ($(HAS_WEBAPP),) | ||
| mkdir -p dist/$(PLUGIN_ID)/webapp/dist; | ||
| cp -r webapp/dist/* dist/$(PLUGIN_ID)/webapp/dist/; | ||
| endif | ||
| cd dist && tar -cvzf $(BUNDLE_NAME) $(PLUGIN_ID) | ||
|
|
||
| @echo plugin built at: dist/$(BUNDLE_NAME) | ||
|
|
||
| ## Builds and bundles the plugin. | ||
| .PHONY: dist | ||
| dist: apply server webapp bundle | ||
|
|
||
| ## Installs the plugin to a (development) server. | ||
| .PHONY: deploy | ||
| deploy: dist | ||
| ## It uses the API if appropriate environment variables are defined, | ||
| ## or copying the files directly to a sibling mattermost-server directory. | ||
| ifneq ($(and $(MM_SERVICESETTINGS_SITEURL),$(MM_ADMIN_USERNAME),$(MM_ADMIN_PASSWORD),$(HTTP)),) | ||
| @echo "Installing plugin via API" | ||
| (TOKEN=`http --print h POST $(MM_SERVICESETTINGS_SITEURL)/api/v4/users/login login_id=$(MM_ADMIN_USERNAME) password=$(MM_ADMIN_PASSWORD) | grep Token | cut -f2 -d' '` && \ | ||
| http --print b GET $(MM_SERVICESETTINGS_SITEURL)/api/v4/users/me Authorization:"Bearer $$TOKEN" && \ | ||
| http --print b DELETE $(MM_SERVICESETTINGS_SITEURL)/api/v4/plugins/$(PLUGIN_ID) Authorization:"Bearer $$TOKEN" && \ | ||
| http --print b --check-status --form POST $(MM_SERVICESETTINGS_SITEURL)/api/v4/plugins plugin@dist/$(BUNDLE_NAME) Authorization:"Bearer $$TOKEN" && \ | ||
| http --print b POST $(MM_SERVICESETTINGS_SITEURL)/api/v4/plugins/$(PLUGIN_ID)/enable Authorization:"Bearer $$TOKEN" && \ | ||
| http --print b POST $(MM_SERVICESETTINGS_SITEURL)/api/v4/users/logout Authorization:"Bearer $$TOKEN" \ | ||
| ) | ||
| else ifneq ($(and $(MM_SERVICESETTINGS_SITEURL),$(MM_ADMIN_USERNAME),$(MM_ADMIN_PASSWORD),$(CURL)),) | ||
| @echo "Installing plugin via API" | ||
| $(eval TOKEN := $(shell curl -i -X POST $(MM_SERVICESETTINGS_SITEURL)/api/v4/users/login -d '{"login_id": "$(MM_ADMIN_USERNAME)", "password": "$(MM_ADMIN_PASSWORD)"}' | grep Token | cut -f2 -d' ' 2> /dev/null)) | ||
| @curl -s -H "Authorization: Bearer $(TOKEN)" -X DELETE $(MM_SERVICESETTINGS_SITEURL)/api/v4/plugins/$(PLUGIN_ID) > /dev/null | ||
| @curl -s -H "Authorization: Bearer $(TOKEN)" -X POST $(MM_SERVICESETTINGS_SITEURL)/api/v4/plugins -F "plugin=@dist/$(BUNDLE_NAME)" > /dev/null && \ | ||
| curl -s -H "Authorization: Bearer $(TOKEN)" -X POST $(MM_SERVICESETTINGS_SITEURL)/api/v4/plugins/$(PLUGIN_ID)/enable > /dev/null && \ | ||
| echo "OK." || echo "Sorry, something went wrong." | ||
| else ifneq ($(wildcard ../mattermost-server/.*),) | ||
| @echo "Installing plugin via filesystem. Server restart and manual plugin enabling required" | ||
| mkdir -p ../mattermost-server/plugins | ||
| tar -C ../mattermost-server/plugins -zxvf dist/$(BUNDLE_NAME) | ||
| else | ||
| @echo "No supported deployment method available. Install plugin manually." | ||
| endif | ||
|
|
||
| ## Runs any lints and unit tests defined for the server and webapp, if they exist. | ||
| .PHONY: test | ||
| test: server/.depensure webapp/.npminstall | ||
| ifneq ($(HAS_SERVER),) | ||
| cd server && $(GO) test -race -v ./... | ||
| endif | ||
| ifneq ($(HAS_WEBAPP),) | ||
| cd webapp && $(NPM) run fix; | ||
| endif | ||
|
|
||
| ## Creates a coverage report for the server code. | ||
| .PHONY: coverage | ||
| coverage: server/.depensure webapp/.npminstall | ||
| ifneq ($(HAS_SERVER),) | ||
| cd server && $(GO) test -race -coverprofile=coverage.txt ./... | ||
| @cd server && $(GO) tool cover -html=coverage.txt | ||
| endif | ||
|
|
||
| ## Clean removes all build artifacts. | ||
| .PHONY: clean | ||
| clean: | ||
| rm -fr dist/ | ||
| ifneq ($(HAS_SERVER),) | ||
| rm -fr server/dist | ||
| rm -fr server/.depensure | ||
| endif | ||
| ifneq ($(HAS_WEBAPP),) | ||
| rm -fr webapp/.npminstall | ||
| rm -fr webapp/dist | ||
| rm -fr webapp/node_modules | ||
| endif | ||
| rm -fr build/bin/ | ||
|
|
||
| # Help documentatin à la https://marmelab.com/blog/2016/02/29/auto-documented-makefile.html | ||
| help: | ||
| @cat Makefile | grep -v '\.PHONY' | grep -v '\help:' | grep -B1 -E '^[a-zA-Z_.-]+:.*' | sed -e "s/:.*//" | sed -e "s/^## //" | grep -v '\-\-' | sed '1!G;h;$$!d' | awk 'NR%2{printf "\033[36m%-30s\033[0m",$$0;next;}1' | sort |
Oops, something went wrong.