-
Notifications
You must be signed in to change notification settings - Fork 14
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
Warn about locust>=0.99 #70
Conversation
Signed-off-by: Thibaut Le Page <thilp@thilp.net>
Signed-off-by: Thibaut Le Page <thilp@thilp.net>
🤖 zincr found 0 problems , 1 warning
Details on how to resolve are provided below Large CommitsChecks all commits for large additions to a single file. Large commits should be reviewed more carefully for potential copyright and licensing issues This file contains a substantial change, please review to determine if the change comes from an external source and if there are any copyright or licensing issues to be aware of
|
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.
Nice!
How funny that they actually released version 0.99 (and 0.999 too, from what I can see), right after 0.14.6
Yes, I think it's a trick to simulate a "redirect" from |
Explain that
locust>=0.99
is not yet supported by Transformer when running a Transformer-generated locustfile inlocust>=0.99
.Related to #69.
Description
Pending decision and implementation of a longer-term fix (support of v1+, potential deprecation of <0.99), we should mitigate the problem by offering clarity to our users.
Types of Changes
TODO
List of tasks you will do to complete the PR:
Review
Reviewers' checklist:
good tests for these flows?
If this PR rather removes flows, are the obsolete tests removed as well?
technical (in source files) and functional (under
docs/
) documentation.In doubt, refer to https://semver.org.