Skip to content

Comments

test: use recommended pytest-vcr settings#491

Merged
jsstevenson merged 1 commit intomainfrom
vcr-in-ci
Feb 4, 2025
Merged

test: use recommended pytest-vcr settings#491
jsstevenson merged 1 commit intomainfrom
vcr-in-ci

Conversation

@jsstevenson
Copy link
Contributor

@jsstevenson jsstevenson commented Feb 4, 2025

from the docs: https://pytest-vcr.readthedocs.io/en/latest/

When running your tests on CI it's recommended to use the --vcr-record=none option. This way you can make sure that you have committed all the cassettes.

@jsstevenson jsstevenson marked this pull request as ready for review February 4, 2025 01:51
@jsstevenson jsstevenson requested review from a team as code owners February 4, 2025 01:51
@jsstevenson jsstevenson added the priority:low Low priority label Feb 4, 2025
@jsstevenson jsstevenson merged commit 12146db into main Feb 4, 2025
14 checks passed
@jsstevenson jsstevenson deleted the vcr-in-ci branch February 4, 2025 13:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@korikuzma korikuzma korikuzma approved these changes

Assignees

No one assigned

Labels

priority:low Low priority

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@jsstevenson @korikuzma