Skip to content

Fix issue #1: Create Github Actions Workflow for dev and main and publish to pypi#2

Draft
lawwu wants to merge 1 commit intomainfrom
openhands-fix-issue-1
Draft

Fix issue #1: Create Github Actions Workflow for dev and main and publish to pypi#2
lawwu wants to merge 1 commit intomainfrom
openhands-fix-issue-1

Conversation

@lawwu
Copy link
Owner

@lawwu lawwu commented Feb 27, 2025

This pull request fixes #1.

The changes fully address all requirements specified in the issue:

  1. Branch-specific workflows are correctly implemented:

    • dev.yml handles testing/linting on dev branch
    • release.yml handles publishing on main branch
  2. Test and lint requirements are met in dev.yml:

    • Implements pytest with coverage reporting
    • Implements flake8 linting
    • Triggers on both push and PR events to dev branch
    • Tests across multiple Python versions (3.8-3.12)
  3. Publishing requirements are met in release.yml:

    • Triggers on push to main branch
    • Builds package using python-build
    • Validates distribution with twine
    • Publishes to PyPI
  4. Security requirements are met:

    • Uses GitHub secrets for PyPI authentication (PYPI_API_TOKEN)
    • Properly configures token-based authentication for PyPI
  5. Performance optimization is implemented:

    • Both workflows use pip caching via actions/setup-python's cache parameter
    • This will speed up dependency installation across runs

The workflows are properly structured, use modern GitHub Actions features (like v4 of checkout and setup-python), and follow best practices for Python package CI/CD. The implementation is complete and should work as intended once the required secrets are configured.

Automatic fix generated by OpenHands 🙌

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.

Create Github Actions Workflow for dev and main and publish to pypi

2 participants