-
Notifications
You must be signed in to change notification settings - Fork 3k
travis: set hypothesis to v3.88 #9371
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
Conversation
@ARMmbed/mbed-os-tools I suspect that hypothesis broke in their latest update |
4927ad8
to
c8f19b2
Compare
Update to v4.x breaks tools, will need to be addressed separately.
@0xc0170, thank you for your changes. |
c8f19b2
to
be9f9f7
Compare
I removed the testing commit, setting hypothesis package back to <4.x version fixes this issue. |
@@ -107,7 +107,7 @@ matrix: | |||
# Install dependencies | |||
- sudo apt-get install gcc-arm-embedded | |||
- pip install -r requirements.txt | |||
- pip install pytest pylint hypothesis mock coverage coveralls | |||
- pip install pytest pylint hypothesis==3.88.3 mock coverage coveralls |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this is the latest known working version, setting it for now
Set 5.11.2 as it should unblock 5.11.2 RC |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks fine, we should be pinning the major versions of stable APIs anyway!
ci started |
Description
Fixing travis test package that got minor update today and broke our tests
Pull request type
Reviewers