Thanks to visit codestin.com
Credit goes to github.com

Skip to content

upgrade dependencies#91

Merged
JiseungHong merged 1 commit into
mainfrom
beta-test
Nov 24, 2025
Merged

upgrade dependencies#91
JiseungHong merged 1 commit into
mainfrom
beta-test

Conversation

@JiseungHong

Copy link
Copy Markdown
Collaborator
  1. Fixed Dependency Resolution Error (pyproject.toml)

Problem: The workflow failed with resolution-too-deep error when trying to install pr-arena==0.2.9 due to conflicting dependency constraints between firebase-admin 7.x, openai 2.x,
and openhands-ai 0.40.0.

Solution: Updated dependency constraints to allow compatible version ranges:

  • firebase-admin: Changed from ^6.6.0 to >=6.6,<8.0 (allows 7.x versions)
  • openai: Changed from >=1.99.5,<1.109.0 to >=1.99.5,<2.8.0 (allows 2.x versions)
  • Added explicit constraints to prevent conflicts:
    • google-api-python-client = ">=2.164.0,<3.0.0" (required by openhands-ai but removed from firebase-admin 7.x)
    • google-api-core = ">=2.0.0,<3.0.0"
    • protobuf = ">=4.21.6,<5.0.0"
    • urllib3 = ">=2.0.7,<3.0.0"
  1. Fixed Linting Error (resolver/analysis/get_data_from_firebase.py)

Problem: Bare except: clause at line 186 violated ruff's E722 rule.

Solution: Changed except: to except Exception: to explicitly catch exceptions.

  1. Updated Workflow to Install from Branch (.github/workflows/pr-arena-resolver.yml)

Problem: Workflow installs from PyPI (pr-arena==0.2.9) which still has the old dependency constraints.

Solution: Changed installation command from:
pip install pr-arena==0.2.9
to:
pip install git+https://github.com/neulab/pr-arena.git@beta-test

This allows the workflow to use the fixed dependencies from the beta-test branch without requiring a new PyPI release.

  1. Regenerated Lock File (poetry.lock)

Updated poetry.lock to reflect the new dependency constraints and ensure reproducible builds.


Verification:

  • ✅ poetry lock completes successfully (no dependency conflicts)
  • ✅ All pre-commit checks pass (ruff, mypy)
  • ✅ All tests pass (90 passed, 1 skipped)
  • ✅ Package builds successfully with poetry build

@JiseungHong JiseungHong merged commit 0871a8f into main Nov 24, 2025
2 checks passed
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.

1 participant