Skip to content
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

Add Abseil from Chromium 114.0.5735.331 #2189

Merged
merged 10 commits into from
Jan 19, 2024

Conversation

jellefoks
Copy link
Member

@jellefoks jellefoks commented Jan 10, 2024

This adds abseil (continuation/replacement of #2179).

b/319139474
This prepares for b/315503544 (and others).

Copy link

codecov bot commented Jan 10, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (7153cc9) 58.53% compared to head (8ddb00f) 58.55%.
Report is 12 commits behind head on main.

❗ Current head 8ddb00f differs from pull request most recent head 613cf30. Consider uploading reports for the commit 613cf30 to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2189      +/-   ##
==========================================
+ Coverage   58.53%   58.55%   +0.01%     
==========================================
  Files        1908     1907       -1     
  Lines       94585    94568      -17     
==========================================
+ Hits        55369    55377       +8     
+ Misses      39216    39191      -25     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@jellefoks jellefoks force-pushed the feature/add-abseil branch 11 times, most recently from 6abc2e6 to 976244d Compare January 12, 2024 01:03
Copy link
Contributor

@andrewsavage1 andrewsavage1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Once this is ready, we should add the subtree squash commit to ensure git history is correct

third_party/chromium/media/BUILD.gn Outdated Show resolved Hide resolved
third_party/abseil-cpp/absl/synchronization/BUILD.gn Outdated Show resolved Hide resolved
third_party/boringssl/BUILD.generated.gni Outdated Show resolved Hide resolved
third_party/abseil-cpp/absl/base/internal/raw_logging.cc Outdated Show resolved Hide resolved
git-subtree-dir: third_party/abseil-cpp
git-subtree-split: fed499399d3f44d3a7957549d493bf30a5d8c867
@andrewsavage1
Copy link
Contributor

LGTM, but I believe the subtree squash commit needs to be added. @dahlstrom-g would you please also take a look?

@dahlstrom-g dahlstrom-g changed the title Add abseil at Chrome tag m114.0.5735.331 Add Abseil from Chromium 114.0.5735.331 Jan 17, 2024
@dahlstrom-g dahlstrom-g force-pushed the feature/add-abseil branch 3 times, most recently from ae31e54 to 290dc80 Compare January 17, 2024 04:00
@dahlstrom-g dahlstrom-g merged commit b04cac7 into youtube:main Jan 19, 2024
330 of 332 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants