Skip to content

Commit a721fc9

Browse files
authored
[feat] makefile adding an aprooval step for version bump targets (#34)
Ticket: https://mattermost.atlassian.net/browse/CLD-7957
1 parent 57b2fcb commit a721fc9

File tree

1 file changed

+14
-0
lines changed

1 file changed

+14
-0
lines changed

Makefile

+14
Original file line numberDiff line numberDiff line change
@@ -49,6 +49,14 @@ define check_pending_pulls
4949
exit 1; \
5050
fi
5151
endef
52+
# Prompt for approval
53+
define prompt_approval
54+
@read -p "About to bump $(APP_NAME) to version $(1), approve? (y/n) " userinput; \
55+
if [ "$$userinput" != "y" ]; then \
56+
echo "Bump aborted."; \
57+
exit 1; \
58+
fi
59+
endef
5260
# ====================================================================================
5361

5462
.PHONY: patch minor major patch-rc minor-rc major-rc
@@ -57,6 +65,7 @@ patch: ## to bump patch version (semver)
5765
$(call check_protected_branch)
5866
$(call check_pending_pulls)
5967
@$(eval PATCH := $(shell echo $$(($(PATCH)+1))))
68+
$(call prompt_approval,$(MAJOR).$(MINOR).$(PATCH))
6069
@echo Bumping $(APP_NAME) to Patch version $(MAJOR).$(MINOR).$(PATCH)
6170
git tag -s -a v$(MAJOR).$(MINOR).$(PATCH) -m "Bumping $(APP_NAME) to Patch version $(MAJOR).$(MINOR).$(PATCH)"
6271
git push origin v$(MAJOR).$(MINOR).$(PATCH)
@@ -67,6 +76,7 @@ minor: ## to bump minor version (semver)
6776
$(call check_pending_pulls)
6877
@$(eval MINOR := $(shell echo $$(($(MINOR)+1))))
6978
@$(eval PATCH := 0)
79+
$(call prompt_approval,$(MAJOR).$(MINOR).$(PATCH))
7080
@echo Bumping $(APP_NAME) to Minor version $(MAJOR).$(MINOR).$(PATCH)
7181
git tag -s -a v$(MAJOR).$(MINOR).$(PATCH) -m "Bumping $(APP_NAME) to Minor version $(MAJOR).$(MINOR).$(PATCH)"
7282
git push origin v$(MAJOR).$(MINOR).$(PATCH)
@@ -78,6 +88,7 @@ major: ## to bump major version (semver)
7888
$(eval MAJOR := $(shell echo $$(($(MAJOR)+1))))
7989
$(eval MINOR := 0)
8090
$(eval PATCH := 0)
91+
$(call prompt_approval,$(MAJOR).$(MINOR).$(PATCH))
8192
@echo Bumping $(APP_NAME) to Major version $(MAJOR).$(MINOR).$(PATCH)
8293
git tag -s -a v$(MAJOR).$(MINOR).$(PATCH) -m "Bumping $(APP_NAME) to Major version $(MAJOR).$(MINOR).$(PATCH)"
8394
git push origin v$(MAJOR).$(MINOR).$(PATCH)
@@ -87,6 +98,7 @@ patch-rc: ## to bump patch release candidate version (semver)
8798
$(call check_protected_branch)
8899
$(call check_pending_pulls)
89100
@$(eval RC := $(shell echo $$(($(RC)+1))))
101+
$(call prompt_approval,$(MAJOR).$(MINOR).$(PATCH)-rc$(RC))
90102
@echo Bumping $(APP_NAME) to Patch RC version $(MAJOR).$(MINOR).$(PATCH)-rc$(RC)
91103
git tag -s -a v$(MAJOR).$(MINOR).$(PATCH)-rc$(RC) -m "Bumping $(APP_NAME) to Patch RC version $(MAJOR).$(MINOR).$(PATCH)-rc$(RC)"
92104
git push origin v$(MAJOR).$(MINOR).$(PATCH)-rc$(RC)
@@ -98,6 +110,7 @@ minor-rc: ## to bump minor release candidate version (semver)
98110
@$(eval MINOR := $(shell echo $$(($(MINOR)+1))))
99111
@$(eval PATCH := 0)
100112
@$(eval RC := 1)
113+
$(call prompt_approval,$(MAJOR).$(MINOR).$(PATCH)-rc$(RC))
101114
@echo Bumping $(APP_NAME) to Minor RC version $(MAJOR).$(MINOR).$(PATCH)-rc$(RC)
102115
git tag -s -a v$(MAJOR).$(MINOR).$(PATCH)-rc$(RC) -m "Bumping $(APP_NAME) to Minor RC version $(MAJOR).$(MINOR).$(PATCH)-rc$(RC)"
103116
git push origin v$(MAJOR).$(MINOR).$(PATCH)-rc$(RC)
@@ -110,6 +123,7 @@ major-rc: ## to bump major release candidate version (semver)
110123
@$(eval MINOR := 0)
111124
@$(eval PATCH := 0)
112125
@$(eval RC := 1)
126+
$(call prompt_approval,$(MAJOR).$(MINOR).$(PATCH)-rc$(RC))
113127
@echo Bumping $(APP_NAME) to Major RC version $(MAJOR).$(MINOR).$(PATCH)-rc$(RC)
114128
git tag -s -a v$(MAJOR).$(MINOR).$(PATCH)-rc$(RC) -m "Bumping $(APP_NAME) to Major RC version $(MAJOR).$(MINOR).$(PATCH)-rc$(RC)"
115129
git push origin v$(MAJOR).$(MINOR).$(PATCH)-rc$(RC)

0 commit comments

Comments
 (0)