Skip to content

Commit

Permalink
Fix broken python github actions (#8580)
Browse files Browse the repository at this point in the history
## Describe your changes
- Looks like pytest 8.2 breaks with tornado
- As a result, we should likely pin 8.1 pytest
- pytest-dev/pytest#12263
## GitHub Issue Link (if applicable)

## Testing Plan

- Explanation of why no additional tests are needed
- Unit Tests (JS and/or Python)
- E2E Tests
- Any manual testing needed?

---

**Contribution License Agreement**

By submitting this pull request you agree that all contributions to this
project are made under the Apache 2.0 license.
  • Loading branch information
willhuang1997 authored and kmcgrady committed May 2, 2024
1 parent 7266453 commit a8a5f9b
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion lib/test-requirements.txt
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,9 @@ urllib3>=1.9
# Testing infrastructure dependencies:
hypothesis>=6.17.4
parameterized
pytest
# Pinned for now because AsyncTest doesn't work with pytest 8.2.0
# https://github.com/pytest-dev/pytest/issues/12263
pytest<8.2.0
pytest-cov
requests-mock
testfixtures
Expand Down

0 comments on commit a8a5f9b

Please sign in to comment.