Skip to content

Commit

Permalink
remove makefile target
Browse files Browse the repository at this point in the history
  • Loading branch information
wdbaruni committed Oct 13, 2024
1 parent c121f50 commit d3e7248
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 10 deletions.
8 changes: 2 additions & 6 deletions .github/workflows/swagger-validation.yml
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,6 @@ jobs:
- name: Install Swag
run: |
go install github.com/swaggo/swag/cmd/swag@latest
shell: bash
# Step 3: Verify swag installation
- name: Verify Swag Installation
Expand All @@ -28,23 +27,20 @@ jobs:
echo "Swag is not installed. Please ensure Go is properly configured and Swag is installed."
exit 1
fi
shell: bash
# Step 4: Generate the swagger.json
- name: Generate Swagger file
run: |
make generate-swagger
shell: bash
./scripts/generate-swagger.sh
# Step 5: Compare the newly generated swagger.json with the committed swagger.json
- name: Check for Swagger differences
run: |
git diff --exit-code pkg/swagger/swagger.json
shell: bash
# Step 6: Fail the build if Swagger doesn't match
- name: Fail if Swagger has differences
if: failure()
run: |
echo "Swagger is outdated. Please regenerate it with 'make generate-swagger' and commit the changes."
echo "Swagger is outdated. Please regenerate it with 'bash ./scripts/generate-swagger.sh' and commit the changes."
exit 1
4 changes: 0 additions & 4 deletions Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -474,7 +474,3 @@ release: build-bacalhau
.PHONY: spellcheck-code
spellcheck-code:
cspell lint -c cspell.yaml --quiet "**/*.{go,js,ts,jsx,tsx,md,yml,yaml,json}"

.PHONY: generate-swagger
generate-swagger:
./scripts/generate_swagger.sh

0 comments on commit d3e7248

Please sign in to comment.