Skip to content

Commit 228d837

Browse files
lev-corruptedclaude
andcommitted
Fix Python version requirements (3.10+ for MCP SDK compatibility)
## Problem GitHub Actions CI/CD was failing because: - pyproject.toml specified Python >=3.9 - But MCP SDK requires Python >=3.10 - This caused dependency resolution errors in CI ## Changes ### Code Configuration - pyproject.toml: `requires-python = ">=3.10"` (was 3.9) - .github/workflows/test.yml: Test matrix now ['3.10', '3.11', '3.12'] (removed 3.9) - .github/workflows/test.yml: Lint job uses Python 3.10 (was 3.9) ### Documentation - README.md: Python 3.10+ badge and requirements - CHANGELOG.md: Updated version references - docs/GITHUB_REPO_INFO.md: Python 3.10+ badge - docs/PROJECT_SUMMARY.md: Python 3.10+ requirements - docs/guides/MCP_SETUP_GUIDE.md: Python 3.10+ references ## Impact - ✅ CI/CD will now pass (compatible with MCP SDK) - ✅ Accurate documentation for users - ✅ No breaking changes for existing users on 3.10+ ## Note Python 3.10 is still widely supported and available on all major platforms. 🤖 Generated with Claude Code https://claude.com/claude-code Co-Authored-By: Claude <noreply@anthropic.com>
1 parent 64cf136 commit 228d837

File tree

7 files changed

+12
-12
lines changed

7 files changed

+12
-12
lines changed

.github/workflows/test.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@ jobs:
1313

1414
strategy:
1515
matrix:
16-
python-version: ['3.9', '3.10', '3.11', '3.12']
16+
python-version: ['3.10', '3.11', '3.12']
1717

1818
steps:
1919
- name: Checkout code
@@ -51,7 +51,7 @@ jobs:
5151
- name: Set up Python
5252
uses: actions/setup-python@v5
5353
with:
54-
python-version: '3.9'
54+
python-version: '3.10'
5555

5656
- name: Install dependencies
5757
run: |

CHANGELOG.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -42,7 +42,7 @@ This release transforms TradingViewMCPServer into a **production-grade, enterpri
4242

4343
- **CI/CD Pipeline**: GitHub Actions workflow
4444
- Automated testing on push/PR
45-
- Multi-version testing (Python 3.9, 3.10, 3.11, 3.12)
45+
- Multi-version testing (Python 3.10, 3.11, 3.12)
4646
- Code quality checks (black, isort, flake8, mypy)
4747
- Coverage reporting with Codecov integration
4848

README.md

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@
33
> **Production-ready multi-asset trading assistant and Pine Script development server for Claude Desktop**
44
55
[![Tests](https://github.com/lev-corrupted/TradingViewMCPServer/workflows/Tests%20and%20Code%20Quality/badge.svg)](https://github.com/lev-corrupted/TradingViewMCPServer/actions)
6-
[![Python 3.9+](https://img.shields.io/badge/python-3.9+-blue.svg)](https://www.python.org/downloads/)
6+
[![Python 3.10+](https://img.shields.io/badge/python-3.10+-blue.svg)](https://www.python.org/downloads/)
77
[![License: MIT](https://img.shields.io/badge/License-MIT-yellow.svg)](https://opensource.org/licenses/MIT)
88
[![Docker](https://img.shields.io/badge/docker-ready-blue.svg)](https://hub.docker.com)
99
[![Version](https://img.shields.io/badge/version-3.4.0-green.svg)](https://github.com/lev-corrupted/TradingViewMCPServer/releases)
@@ -18,7 +18,7 @@ Professional trading assistant supporting **Forex, Stocks, and Crypto** with **2
1818
- **🔄 Auto-Retry Logic**: Exponential backoff for network failures (3 retries: 2s, 4s, 8s)
1919
- **⚡ LRU Cache**: Memory-bounded cache (1000 entries) with automatic eviction
2020
- **🐳 Docker Support**: Production-ready containerization with Docker Compose
21-
- **🚀 CI/CD Pipeline**: Automated testing across Python 3.9-3.12
21+
- **🚀 CI/CD Pipeline**: Automated testing across Python 3.10-3.12
2222
- **✅ 100% Test Coverage**: 44/44 tests passing
2323

2424
### 📊 Trading & Market Analysis
@@ -45,7 +45,7 @@ Professional trading assistant supporting **Forex, Stocks, and Crypto** with **2
4545

4646
### Prerequisites
4747

48-
- Python 3.9+ or Docker
48+
- Python 3.10+ or Docker
4949
- Claude Desktop
5050
- Alpha Vantage API key ([Get free key](https://www.alphavantage.co/support/#api-key))
5151

@@ -325,7 +325,7 @@ This version included major improvements and refactoring:
325325
- **Test Suite**: pytest-based testing framework included
326326

327327
### Development
328-
- **Lower Python Requirement**: Now works with Python 3.9+ (was 3.13+)
328+
- **Lower Python Requirement**: Now works with Python 3.10+ (was 3.13+)
329329
- **Development Tools**: Added black, mypy, pytest, and other dev dependencies
330330
- **Better Documentation**: Improved docstrings and code comments
331331

docs/GITHUB_REPO_INFO.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -42,7 +42,7 @@ Add these to the top of README.md (already added):
4242

4343
```markdown
4444
[![Tests](https://github.com/lev-corrupted/TradingViewMCPServer/workflows/Tests%20and%20Code%20Quality/badge.svg)](https://github.com/lev-corrupted/TradingViewMCPServer/actions)
45-
[![Python 3.9+](https://img.shields.io/badge/python-3.9+-blue.svg)](https://www.python.org/downloads/)
45+
[![Python 3.10+](https://img.shields.io/badge/python-3.9+-blue.svg)](https://www.python.org/downloads/)
4646
[![License: MIT](https://img.shields.io/badge/License-MIT-yellow.svg)](https://opensource.org/licenses/MIT)
4747
[![Docker](https://img.shields.io/badge/docker-ready-blue.svg)](https://hub.docker.com)
4848
[![Version](https://img.shields.io/badge/version-3.4.0-green.svg)](https://github.com/lev-corrupted/TradingViewMCPServer/releases)

docs/PROJECT_SUMMARY.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -303,7 +303,7 @@ flake8 tradingview_mcp/ # Lint
303303
- **Version Detection**: See [PINE_SCRIPT.md](PINE_SCRIPT.md)
304304
- **Error Explanations**: Use `explain_pine_error()` tool
305305
- **API Rate Limits**: Check cache settings in code
306-
- **Installation Issues**: Verify Python 3.9+ and virtual environment
306+
- **Installation Issues**: Verify Python 3.10+ and virtual environment
307307

308308
---
309309

docs/guides/MCP_SETUP_GUIDE.md

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -163,7 +163,7 @@ This guide documents all MCP servers configured for optimal development workflow
163163

164164
**Use Cases:**
165165
```
166-
"Remember that we're using Python 3.9+ for this project"
166+
"Remember that we're using Python 3.10+ for this project"
167167
"What did we decide about the architecture?"
168168
"Recall our discussion about Pine Script v6"
169169
"Store this pattern for future reference"
@@ -326,7 +326,7 @@ Expected response should list:
326326

327327
**Memory:**
328328
```
329-
"Remember that this project uses Python 3.9+"
329+
"Remember that this project uses Python 3.10+"
330330
```
331331

332332
**Git:**

pyproject.toml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@ name = "tradingview-mcp"
33
version = "3.3.0"
44
description = "TradingView MCP server for Claude - Multi-asset trading assistant with Pine Script v6 development tools supporting Forex, Stocks, Crypto with 25+ technical indicators"
55
readme = "README.md"
6-
requires-python = ">=3.9"
6+
requires-python = ">=3.10"
77
dependencies = [
88
"mcp[cli]>=1.12.0",
99
"python-dotenv>=1.0.0",

0 commit comments

Comments
 (0)