Compare commits

..

2 commits

Author SHA1 Message Date
Ori Bruk
ce37ce12d6 [#68] Ensure compatibility of different API versions with each other.
All checks were successful
Formatters / Run fmt (pull_request) Successful in 24s
Pre-commit hooks / Pre-commit (pull_request) Successful in 36s
DCO action / DCO (pull_request) Successful in 42s
Update linters.

Signed-off-by: Ori Bruk <o.bruk@yadro.com>
2024-11-06 15:01:19 +03:00
Ori Bruk
2466ce876e [#68] Remove unnecessary language options.
Change proto package path.

Signed-off-by: Ori Bruk <o.bruk@yadro.com>
2024-11-06 14:59:06 +03:00

View file

@ -1,6 +1,8 @@
#!/usr/bin/make -f
SHELL=bash
VERSION_CHECK_BRANCH ?=$(shell git describe --tags --match "version-check*" 2>/dev/null || echo "remotes/origin/master")
include help.mk
.PHONY: doc fmt check-version compatibility pre-commit unpre-commit pre-commit-run
@ -24,8 +26,8 @@ fmt:
# Run version update check in version.json file
check-version:
@protoChanges=$$(git diff --name-only remotes/origin/master | grep '**/*.proto'); \
version=$$(git show remotes/origin/master:version.json); \
@protoChanges=$$(git diff --name-only $(VERSION_CHECK_BRANCH) | grep '**/*.proto'); \
version=$$(git show $(VERSION_CHECK_BRANCH):version.json); \
if [ -z "$$version" ] || [ -z "$$protoChanges" ]; then \
exit; \
fi; \
@ -36,24 +38,32 @@ check-version:
if [ "$$masterMilestone" -eq "$$milestone" -a "$$masterPatch" -eq "$$patch" ]; then \
echo "⇒ You need to update the api version in the version.json file"; \
exit 1; \
elif [ "$$masterMilestone" -gt "$$milestone" ]; then \
fi; \
if [ "$$masterMilestone" -gt "$$milestone" ]; then \
echo "⇒ The milestone cannot be decrement in the version.json file"; \
exit 1; \
elif [ "$$masterMilestone" -eq "$$milestone" -a "$$masterPatch" -gt "$$patch" ]; then \
fi; \
if [ "$$masterMilestone" -eq "$$milestone" -a "$$masterPatch" -gt "$$patch" ]; then \
echo "⇒ The patch cannot be decrement without milestone increment in the version.json file"; \
exit 1; \
elif [ "$$masterMilestone" -lt "$$milestone" -a "$$patch" -ne 0 ]; then \
fi; \
if [ "$$masterMilestone" -lt "$$milestone" -a "$$patch" -ne 0 ]; then \
echo "⇒ The patch should be 0 after milestone increment in the version.json file"; \
exit 1; \
fi
# Run a compatibility information check \
Checks that the files that describe compatibility in the compatibility package \
have the current version information added in the version.json file
# Run a compatibility information check
# Checks that the files that describe compatibility in the compatibility package
# have the current version information added in the version.json file.
# Description of logic:
# 1. Determine the current API version from the version.json file;
# 2. Find all JSON files with compatibility descriptions in the compatibility package;
# 3. For each such file, we define all the described rpc methods;
# 4. For each rpc method in each compatibility file, we check that the updated version information is present.
compatibility:
@version=$$(jq -r '.milestone' version.json).$$(jq -r '.patch' version.json); \
conflicts=0; \
for f in `ls **/*.json`; do \
for f in `ls compatibility/*.json`; do \
for rpc in $$(jq -r '.[] | to_entries | .[].key' $$f); do \
if $$(jq -r '.[].'$$rpc' | has("v'$$version'") | not' $$f); then \
echo "⇒ No compatibility information found for version" $$version "in file" $$f "for rpc" $$rpc; \