Skip to content

Add Python build configuration and packaging setup#5

Draft
Copilot wants to merge 3 commits intomasterfrom
copilot/initial-setup
Draft

Add Python build configuration and packaging setup#5
Copilot wants to merge 3 commits intomasterfrom
copilot/initial-setup

Conversation

Copy link

Copilot AI commented Jan 25, 2026

The project lacked standard Python packaging infrastructure, requiring manual dependency installation and preventing pip-based workflows.

Changes

Build Configuration

  • requirements.txt - declares requests dependency
  • setup.py - enables pip install -e . with package metadata
  • requirements-dev.txt - adds pytest, flake8, black, mypy for development
  • MANIFEST.in - excludes database files from distributions

Package Structure

  • app/__init__.py - makes app a discoverable package for setuptools

Docker & Ignores

  • Dockerfile - uses requirements.txt for better layer caching
  • .gitignore - excludes __pycache__, *.egg-info, dist/, build/, virtual environments

Usage

# Install project
pip install -e .

# Install with dev dependencies
pip install -r requirements-dev.txt

# Build Docker image (now caches dependency layer)
docker build -t binance-trader .

Dependencies verified against GitHub Advisory Database - no vulnerabilities found.

Original prompt

Pull Request: mdellison90-stack/bbchop#1


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Copilot AI and others added 2 commits January 25, 2026 11:38
Co-authored-by: mdellison90-stack <230609064+mdellison90-stack@users.noreply.github.com>
Co-authored-by: mdellison90-stack <230609064+mdellison90-stack@users.noreply.github.com>
Copilot AI changed the title [WIP] Add initial project setup and configuration Add Python build configuration and packaging setup Jan 25, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants