Generated by 🚫 dangerJS against 739e168

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 82.52%. Comparing base (b24b30c) to head (739e168).
Report is 1 commits behind head on master.

@@            Coverage Diff             @@
##           master    #1110      +/-   ##
==========================================
- Coverage   82.58%   82.52%   -0.07%     
==========================================
  Files          53       53              
  Lines        7896     7896              
  Branches     1234     1234              
==========================================
- Hits         6521     6516       -5     
- Misses       1265     1268       +3     
- Partials      110      112       +2     

LGTM

Great! Happy New Year, @JoshuaMoelans.

I want to review the changes needed in the forks line-by-line with you before we merge anything. A considerable amount of C++17 and 20 specifics require changing, and I want to ensure these are clear. We also must find a getsentry home for the mini_chromium fork first (since this is currently hosted on my Github).

@JoshuaMoelans, now that we have reviewed it in detail and have a getsentry fork of mini_chromium, can I ask you to have one last look at the submodule chain to double-check for any stupidities on my part? Thx!