Skip to content

Pin the pytest-recording library to resolve a windows build issue#344

Merged
michael-richey merged 4 commits intomainfrom
michael.richey/windows-build-fix
May 7, 2025
Merged

Pin the pytest-recording library to resolve a windows build issue#344
michael-richey merged 4 commits intomainfrom
michael.richey/windows-build-fix

Conversation

@michael-richey
Copy link
Copy Markdown
Collaborator

@michael-richey michael-richey commented May 7, 2025

What does this PR do?

Pins pytest-recording to version 0.13.2, version 0.13.3 introduced an error in our windows build. There's an open issue in that project: kiwicom/pytest-recording#174

Verification Process

Ran our windows build and confirmed it's working again.

Review checklist (to be filled by reviewers)

  • Feature or bug fix MUST have appropriate tests (unit, integration, etc...)
  • PR title must be written as a CHANGELOG entry (see why)
  • Files changes must correspond to the primary purpose of the PR as described in the title (small unrelated changes should have their own PR)
  • PR must have one changelog/ label attached. If applicable it should have the backward-incompatible label attached.
  • PR should not have do-not-merge/ label attached.
  • If Applicable, issue must have kind/ and severity/ labels attached at least.

@michael-richey michael-richey changed the title Start a window-build-fix branch Pin the pytest-recording library to resolve a windows build issue May 7, 2025
@michael-richey michael-richey marked this pull request as ready for review May 7, 2025 16:40
@michael-richey michael-richey requested a review from a team as a code owner May 7, 2025 16:40
@michael-richey michael-richey merged commit d8761ee into main May 7, 2025
17 checks passed
@michael-richey michael-richey deleted the michael.richey/windows-build-fix branch May 7, 2025 18:27
@Stranger6667
Copy link
Copy Markdown

Folks, that was fixed in 0.13.4 - sorry for the trouble

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants