-
Notifications
You must be signed in to change notification settings - Fork 1.3k
Conversation
Codecov Report
@@ Coverage Diff @@
## master #17843 +/- ##
============================================
- Coverage 32.97% 32.96% -0.02%
+ Complexity 1383 1382 -1
============================================
Files 461 461
Lines 19426 19425 -1
Branches 2712 2712
============================================
- Hits 6406 6403 -3
- Misses 12487 12488 +1
- Partials 533 534 +1
Continue to review full report at Codecov.
|
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.
Stamp!
Please rebase, the blocking test was disabled: f0b7b3b |
Do we really want that though? We had travis/buddybuild and others in parallel and it was always triple the pain to have two build systems. |
@pocmo Very good point. We discussed with RelEng and agreed this would be a temporary solution until we can build contributor PRs on on Task Cluster using the same CI (or subset) of what we use. I also included checks in this workflow so that it will only run on forks so it should not get in the way of work happening in the main repo. |
5a4a3ea
to
566261f
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.
Awesome 🎉
8e2f17c
to
49d629a
Compare
Is this a test that requires a device?
Similar ...
|
This is odd?
|
I switched to JDK 1.8 instead of 11 and that seems to have gotten rid of the weird failures (SSL related and those tests for LG and Samsung). What is left are now these:
What do you think @pocmo - these are fine on TaskCluster - what could make these run different on GitHub? |
@pocmo is it possible that these tests depend on a GeoIP lookup and that the GitHub actions run in a different country than Task Cluster? Just a random guess ... |
Fix for failing tests at #18048 - when that lands, rebase this patch and it is ready to land. |
a6e82b0
to
44b76e0
Compare
Experimental GitHub Actions Workflow to build contributor pull requests. I took most of this workflow from https://github.com/fork-maintainers/iceraven-browser and modified it slightly.
TODO Enable the checks to make sure this only runs on forks.