Skip to content

Update python version and README #35

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

Jump to bottom
Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Update python version and README #35

wants to merge 2 commits into from

Conversation

jeffersonBastos
Copy link

Update Python Solver Template - Work in Progress

This PR contains updates to the CoW Protocol Solver Template (Python) as part of ongoing development efforts to modernize the template.

  1. Python Version Update
  2. Documentation Improvements
  3. Docker Support Removal
    • Removed Dockerfile and all Docker-related documentation (docker image outdated)

Development Status:

  • This template is currently under active development and should be considered a work in progress. The changes included in this PR are part of ongoing efforts.

@socket-security Socket Security
Copy link

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security		 Vulnerability Quality Maintenance License
Updatedpydantic@​1.9.0 ⏵ 1.10.1293 +1100100100100
Updateduvicorn@​0.17.6 ⏵ 0.22.098 +1100100100100
Updatedfastapi@​0.65.2 ⏵ 0.95.2100 +1100100100100

View full report

fhenneke
fhenneke approved these changes Sep 18, 2025
View reviewed changes
Copy link
Contributor

@fhenneke fhenneke left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks legit.

Since the current version is not maintained, I am approving without thorough testing.

README.md
2. **Implement Solver Logic**: Modify the solve endpoint in `src/_server.py`
3. **Add Pathfinding**: Implement algorithms to find optimal trading paths
4. **Handle AMMs**: Add support for different AMM protocols
5. **Optimize**: Implement price optimization and MEV protection
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would probably have used 3. AMMS, 4. Pathfinding, no 5. But feel free to ignore.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@fhenneke fhenneke fhenneke approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@jeffersonBastos @fhenneke