Skip to content
This repository has been archived by the owner on Apr 26, 2024. It is now read-only.

update pytest requirements #387

Merged
merged 1 commit into from
Feb 27, 2024
Merged

update pytest requirements #387

merged 1 commit into from
Feb 27, 2024

Conversation

zzstoatzz
Copy link
Contributor

were getting a bunch of AttributeError: 'CallSpec2' object has no attribute 'funcargs' errors because we were using pytest 8.0

@zzstoatzz zzstoatzz requested a review from a team as a code owner February 27, 2024 16:18
@zzstoatzz zzstoatzz self-assigned this Feb 27, 2024
zangell44
zangell44 approved these changes Feb 27, 2024
View reviewed changes
Copy link
Contributor

@zangell44 zangell44 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for tackling this!

@zzstoatzz zzstoatzz merged commit 3481235 into main Feb 27, 2024
6 checks passed
@zzstoatzz zzstoatzz deleted the pin-pytest branch February 27, 2024 17:01
@desertaxle
Copy link
Member

Looks like this issue is coming from pytest-lazy-fixture: TvoroG/pytest-lazy-fixture#65. I can update our use of pytest-lazy-fixture so that we can remove the pytest pin.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@zangell44 zangell44 zangell44 approved these changes

Assignees

@zzstoatzz zzstoatzz

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@zzstoatzz @desertaxle @zangell44