Skip to content

Commit 55cd21d

Browse files
authored
Fixed doctest-check target in Makefile (#442)
Signed-off-by: Manav Gupta <[email protected]>
1 parent e6e4ee6 commit 55cd21d

File tree

1 file changed

+2
-9
lines changed

1 file changed

+2
-9
lines changed

Makefile

Lines changed: 2 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -270,15 +270,8 @@ doctest-check:
270270
@echo "🔍 Checking doctest coverage..."
271271
@test -d "$(VENV_DIR)" || $(MAKE) venv
272272
@/bin/bash -c "source $(VENV_DIR)/bin/activate && \
273-
python3 -c \"import subprocess, sys; \
274-
result = subprocess.run(['python', '-m', 'pytest', '--doctest-modules', 'mcpgateway/', '--tb=no', '-q'], capture_output=True); \
275-
if result.returncode == 0: \
276-
print('✅ All doctests passing'); \
277-
else: \
278-
print('❌ Doctest failures detected'); \
279-
print(result.stdout.decode()); \
280-
print(result.stderr.decode()); \
281-
sys.exit(1)\""
273+
python3 -m pytest --doctest-modules mcpgateway/ --tb=no -q && \
274+
echo '✅ All doctests passing' || (echo '❌ Doctest failures detected' && exit 1)"
282275

283276
# =============================================================================
284277
# 📊 METRICS

0 commit comments

Comments
 (0)