Skip to content

Conversation

@Liam-DeVoe
Copy link
Member

See #4481. Untested - I'm happy to add one if someone sees an easy way to test this.

@jobh
Copy link
Contributor

jobh commented Sep 22, 2025

Untested - I'm happy to add one if someone sees an easy way to test this.

Maybe add a setenv = CI_internal={env:CI} to tox and a test that is_in_ci() is true if CI_internal is set?

@Liam-DeVoe
Copy link
Member Author

tempting..I'm wary of tying our tests to our test runner so tightly though

@Liam-DeVoe Liam-DeVoe closed this Sep 22, 2025
@Liam-DeVoe Liam-DeVoe reopened this Sep 22, 2025
@Liam-DeVoe Liam-DeVoe merged commit 3b95a47 into HypothesisWorks:master Sep 22, 2025
78 of 81 checks passed
@Liam-DeVoe Liam-DeVoe deleted the tox-ci branch September 23, 2025 14:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants