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

remove hypothesis (at least for now) #583

Merged
merged 9 commits into from
May 2, 2023
Merged

Conversation

glyph
Copy link
Member

@glyph glyph commented May 2, 2023

oops #586

@glyph glyph requested a review from a team as a code owner May 2, 2023 21:00
@codecov
Copy link

codecov bot commented May 2, 2023

Codecov Report

Patch coverage: 100.00% and project coverage change: +0.01 🎉

Comparison is base (9151592) 99.04% compared to head (c555bfb) 99.05%.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #583      +/-   ##
==========================================
+ Coverage   99.04%   99.05%   +0.01%     
==========================================
  Files          44       45       +1     
  Lines        3887     3935      +48     
  Branches      520      523       +3     
==========================================
+ Hits         3850     3898      +48     
  Misses         23       23              
  Partials       14       14              
Impacted Files Coverage Δ
src/klein/test/__init__.py 100.00% <ø> (ø)
src/klein/test/not_hypothesis.py 100.00% <100.00%> (ø)
src/klein/test/test_headers.py 100.00% <100.00%> (ø)
src/klein/test/test_message.py 100.00% <100.00%> (ø)
src/klein/test/test_plating.py 100.00% <100.00%> (ø)
src/klein/test/test_request_compat.py 100.00% <100.00%> (ø)

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@glyph
Copy link
Member Author

glyph commented May 2, 2023

Worth noting that this cuts time spent in CI in half without affecting coverage.

image

@glyph glyph changed the base branch from master to pre-commit-ci-update-config May 2, 2023 22:01
@glyph glyph merged commit 9d91f10 into pre-commit-ci-update-config May 2, 2023
@glyph glyph deleted the falsified branch May 2, 2023 22:01
@glyph glyph restored the falsified branch May 2, 2023 22:01
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.

hyperlink generates invalid URLs via hypothesis which sometimes causes our tests to fail
1 participant