Skip to content

feat(test): test on multiple web3 major versions#189

Open
BobTheBuidler wants to merge 3 commits intomasterfrom
codex/update-testing-workflow-for-web3.py-versions
Open

feat(test): test on multiple web3 major versions#189
BobTheBuidler wants to merge 3 commits intomasterfrom
codex/update-testing-workflow-for-web3.py-versions

Conversation

@BobTheBuidler
Copy link
Collaborator

Motivation

  • The workflow explicitly forced web3>=7,<8 for the web3-major=7 matrix entry which is unnecessary when poetry/the lockfile should control installed versions.
  • The poetry.lock currently lists web3 = "6.11.0", showing the lockfile already pins a web3 version and the v7 override was redundant.
  • Removing the override simplifies the Install web3.py step and avoids clobbering poetry's installation choices.
  • Keep full matrix coverage for web3-major while letting poetry/the lockfile determine the actual package installed.

Description

  • Updated .github/workflows/pytest.yaml to remove the else branch that ran poetry run pip install "web3>=7,<8".
  • The Install web3.py step now only installs pinned ranges for web3-major 5 and 6 and leaves 7 to the lockfile/default install path.
  • This change removes two lines and the conditional override in the workflow's Install web3.py step.

Testing

  • No automated tests were run because this change only modifies the CI workflow.

Codex Task

@BobTheBuidler BobTheBuidler changed the title Remove web3 v7 install override from pytest workflow feat(test): test on multiple web3 versions Jan 13, 2026
@BobTheBuidler BobTheBuidler changed the title feat(test): test on multiple web3 versions feat(test): test on multiple web3 major versions Jan 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant