-
Notifications
You must be signed in to change notification settings - Fork 84
New CI fixes #1185
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
New CI fixes #1185
Conversation
The newer pyparsing versions no differs on the usage of __bool__ so we can check the list instead.
Do not merge, pushed commit to test pyparsing change with pyparsing <3.0.0 versions. |
Verified with pyparsing==2.4.7 - reverting test changes. |
@@ -1,26 +1,46 @@ | |||
queue_rules: | |||
- name: default | |||
conditions: |
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.
if they are identical to condition to be put in queue you don't need to repeat them fwiw.
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.
I'd prefer 2 PR but ok :)
I'm not clicking merge but you can (and need to) merge manually @tobias-urdin |
@Mergifyio backport stable/4.4 |
✅ Backports have been created
|
New CI fixes (backport #1185)
@Mergifyio backport stable/4.3 |
✅ Backports have been created
|
This contains new CI fixes.
__bool__
on pyparsing.ParseResult classstrict
key in themerge
dict