-
Notifications
You must be signed in to change notification settings - Fork 666
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
fixes #1697 shard travis builds for python2 #1698
fixes #1697 shard travis builds for python2 #1698
Conversation
NB: if it appears to be impossible, there is already some ticket to propose some more deep enhancement #1575 |
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.
Sorry, I had to be strict in this review.
@webknjaz Thank you for your comments, much appreciated. I replied to some, and I will adapt after you confirm the final direction to take. |
Signed-off-by: Fabrice Flore-Thebault <themr0c@users.noreply.github.com>
Signed-off-by: Fabrice Flore-Thebault <themr0c@users.noreply.github.com>
Signed-off-by: Fabrice Flore-Thebault <themr0c@users.noreply.github.com>
Signed-off-by: Fabrice Flore-Thebault <themr0c@users.noreply.github.com>
Signed-off-by: Fabrice Flore-Thebault <themr0c@users.noreply.github.com>
Signed-off-by: Fabrice Flore-Thebault <themr0c@users.noreply.github.com>
9664c45
to
206ce1e
Compare
Signed-off-by: Fabrice Flore-Thebault <themr0c@users.noreply.github.com>
Good news the errors on the last build https://travis-ci.com/ansible/molecule/builds/98812943 are errors that are already in master, unrelated to this PR. |
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.
Also with so many changes this thing is unreviewable
This reverts commit e8c5369.
…o keep the PR readable. Signed-off-by: Fabrice Flore-Thebault <themr0c@users.noreply.github.com>
👍 |
* fixes ansible#1697 shard travis builds for python2 Signed-off-by: Fabrice Flore-Thebault <themr0c@users.noreply.github.com> * fixes ansible#1697 shard travis builds for python2 Signed-off-by: Fabrice Flore-Thebault <themr0c@users.noreply.github.com> * attempt to shard tests on py27 without breaking tests on py3 Signed-off-by: Fabrice Flore-Thebault <themr0c@users.noreply.github.com> * attempt to shard tests on py27 without breaking tests on py3 Signed-off-by: Fabrice Flore-Thebault <themr0c@users.noreply.github.com> * attempt to shard tests on py27 without breaking tests on py3 Signed-off-by: Fabrice Flore-Thebault <themr0c@users.noreply.github.com> * attempt to shard tests on py27 without breaking tests on py3 Signed-off-by: Fabrice Flore-Thebault <themr0c@users.noreply.github.com> * attempt to shard tests on py27 without breaking tests on py3 Signed-off-by: Fabrice Flore-Thebault <themr0c@users.noreply.github.com> * fix yaml, fix yamllint errors * Revert "fix yaml, fix yamllint errors" This reverts commit e8c5369. * Fix yaml syntax. Keep wrong indentation errors reported by yamllint to keep the PR readable. Signed-off-by: Fabrice Flore-Thebault <themr0c@users.noreply.github.com>
Signed-off-by: Fabrice Flore-Thebault themr0c@users.noreply.github.com
fixes #1697 shard travis builds for python2
PR Type