Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

CI Test: Deprecating 'Healthcheck.all()' from Hypothesis in fuzz.py #3945

Merged
merged 1 commit into from Oct 16, 2023

Conversation

rdrll
Copy link
Contributor

@rdrll rdrll commented Oct 16, 2023

Description

This PR replaced the deprecating Healthcheck.all() method used in the Fuzz GitHub Action with latest Hypothesis
supporting approach.

See their 6.72.0 release about this deprecation.

Hypothesis has deprecated this method in April, considering they usually remove features after half year of deprecation, it seems appropriate to replace this usage for us as well.

After this fix, there will be no more HypothesisDeprecationWarning like this:

image

Checklist - did you ...

  • Add an entry in CHANGES.md if necessary? not necessary
  • Add / update tests if necessary? not necessary
  • Add new / update outdated documentation? not necessary

Copy link
Collaborator

@hauntsaninja hauntsaninja left a comment

Choose a reason for hiding this comment

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

Thanks!

@hauntsaninja hauntsaninja added the skip news Pull requests that don't need a changelog entry. label Oct 16, 2023
@JelleZijlstra JelleZijlstra merged commit 3bb9214 into psf:main Oct 16, 2023
38 of 39 checks passed
@rdrll rdrll deleted the hypothesis_deprecate_method branch October 16, 2023 17:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
skip news Pull requests that don't need a changelog entry.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants