Skip to content

fix: align ci lint command with current ruff syntax #24

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
itprodirect wants to merge 1 commit into
base: main
Choose a base branch
from
Open

fix: align ci lint command with current ruff syntax #24

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 6 additions & 0 deletions .github/workflows/ci.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -18,6 +18,12 @@ jobs:
run: |
python -m pip install --upgrade pip
if [ -f requirements.txt ]; then pip install -r requirements.txt; fi
- name: Lint with ruff
run: |
ruff check .
- name: Type check with mypy
run: |
mypy --strict .
- name: Run tests
run: |
if [ -d tests ]; then pytest -q; else echo "No tests"; fi
10 changes: 6 additions & 4 deletions README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
# 🚀 Model-Context-Protocol-101

[![Python](https://img.shields.io/badge/Python-3.8%2B-blue.svg)](https://www.python.org/)
[![Python](https://img.shields.io/badge/Python-3.10%2B-blue.svg)](https://www.python.org/)
[![License](https://img.shields.io/badge/License-MIT-green.svg)](https://opensource.org/licenses/MIT)
[![Build Status](https://img.shields.io/github/actions/workflow/status/itprodirect/Model-Context-Protocol-101/ci.yml)](https://github.com/itprodirect/Model-Context-Protocol-101/actions)
[![Dependencies](https://img.shields.io/badge/Dependencies-Updated-brightgreen.svg)](https://github.com/itprodirect/Model-Context-Protocol-101/blob/main/requirements.txt)
Expand Down Expand Up @@ -47,6 +47,8 @@ venv\Scripts\activate
```bash
pip install -r requirements.txt
```
This installs both third-party libraries and the local `mcp101` package, so the
CLI and utilities can be imported from anywhere in the project.
<div style="background-color:#fff7e6;border-left:4px solid #ff8c00;padding:10px;margin:10px 0;">
<strong>Time Saver:</strong> A single command installs everything needed so independent agents can start experimenting right away.
</div>
Expand Down Expand Up @@ -112,10 +114,10 @@ Use the command-line interface to run common tasks directly from the terminal.

```bash
# Calculate profit from revenue and cost
python src/cli.py profit 1000 600
mcp101-cli profit 1000 600

# Total commission from the sample dataset
python src/cli.py commission data/insurance_sales.csv
mcp101-cli commission data/insurance_sales.csv
```

---
Expand Down Expand Up @@ -159,7 +161,7 @@ For questions or collaborations, connect with me on **LinkedIn** or open an **Is
**Virtual environment won't activate**
Make sure you run `python -m venv venv` and then activate it with
`source venv/bin/activate` on Mac/Linux or `venv\Scripts\activate` on Windows.
Verify Python 3.8+ is installed.
Verify Python 3.10+ is installed.

**Missing packages**
Run `pip install -r requirements.txt` from the project root while your virtual
Expand Down
Loading