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

Remove polyfill.io reference #4031

Merged
merged 1 commit into from
Aug 21, 2024

Conversation

andrewsavage1
Copy link
Contributor

@andrewsavage1 andrewsavage1 commented Aug 20, 2024

b/361119691

@andrewsavage1
Copy link
Contributor Author

Don't imagine this would affect devtools as we don't currently have lighthouse available but trying it out now

@andrewsavage1 andrewsavage1 marked this pull request as ready for review August 21, 2024 01:58
@andrewsavage1
Copy link
Contributor Author

Tested devtools locally—we didn't have a lighthouse tab in devtools before, and we definitely don't have one with this change. Doesn't seem to affect any other tabs, tried out a trace and that went fine.

@andrewsavage1 andrewsavage1 merged commit 0fa8584 into youtube:main Aug 21, 2024
313 of 315 checks passed
@andrewsavage1 andrewsavage1 deleted the fix/remove-polyfill branch August 21, 2024 01:59
@andrewsavage1 andrewsavage1 added cp-23.lts.1+ Cherry Pick to the 23.lts.1+ branch cp-22.lts.1+ Cherry Pick to the 22.lts.1+ branch cp-24.lts.1+ Cherry Pick to the 24.lts.1+ branch cp-25.lts.1+ Cherry Pick to the 25.lts.1+ branch labels Aug 21, 2024
cobalt-github-releaser-bot pushed a commit that referenced this pull request Aug 21, 2024
b/361119691

(cherry picked from commit 0fa8584)
cobalt-github-releaser-bot pushed a commit that referenced this pull request Aug 21, 2024
b/361119691

(cherry picked from commit 0fa8584)
cobalt-github-releaser-bot pushed a commit that referenced this pull request Aug 21, 2024
b/361119691

(cherry picked from commit 0fa8584)
cobalt-github-releaser-bot pushed a commit that referenced this pull request Aug 21, 2024
b/361119691

(cherry picked from commit 0fa8584)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cp-22.lts.1+ Cherry Pick to the 22.lts.1+ branch cp-23.lts.1+ Cherry Pick to the 23.lts.1+ branch cp-24.lts.1+ Cherry Pick to the 24.lts.1+ branch cp-25.lts.1+ Cherry Pick to the 25.lts.1+ branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants