-
-
Notifications
You must be signed in to change notification settings - Fork 75
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
Suppress webm file in media content e2e test #1944
Suppress webm file in media content e2e test #1944
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #1944 +/- ##
==========================================
+ Coverage 73.27% 76.34% +3.07%
==========================================
Files 39 37 -2
Lines 3106 2985 -121
Branches 687 653 -34
==========================================
+ Hits 2276 2279 +3
+ Misses 711 611 -100
+ Partials 119 95 -24
☔ View full report in Codecov by Sentry. |
What is the explanation about:
|
@kelson42 I updated PR description. |
b0df307
to
0e7d59a
Compare
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.
Approved, but there is a real bug somehwere either in Mediawiki or in MWoffliner
@VadimKovalenkoSNF please fix conflict and merge |
5bd9260
to
476f29d
Compare
476f29d
to
5c45741
Compare
Workaround for #1943
Problem:
vp9.webm
file is no longer present in the I namespace, probably it shouldn't.Workaround:
Temporarily disabled any
vp9.webm
checks till the root cause is found.