-
-
Notifications
You must be signed in to change notification settings - Fork 2.7k
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
Merge master into features #5332
Conversation
Correct some tiny typos in changelog
Temporarily pin pluggy on master
conftest: add uses_pexpect mark
tests: conftest: fix collection of slow/slowest items
(dropping the temporary pinning of pluggy (2e2e895))
I've removed 2e2e895 in the merge commit - or should it be reverted instead? |
Codecov Report
@@ Coverage Diff @@
## features #5332 +/- ##
============================================
- Coverage 92.87% 88.23% -4.64%
============================================
Files 115 115
Lines 26264 26430 +166
Branches 2595 2639 +44
============================================
- Hits 24392 23321 -1071
- Misses 1548 2769 +1221
- Partials 324 340 +16
Continue to review full report at Codecov.
|
Probably reverted, I suspect if one attempts to merge it again, that commit will again be a merge candidate |
A revert would be cleaner (it would also be reverted on master when merging it back), but that also works with the merge commit - where I've done it initially since it caused a conflict right there. I think it is good like this. |
No description provided.