-
Notifications
You must be signed in to change notification settings - Fork 3.9k
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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
馃彈 Remove gulp
streaming from a few developer tasks
#32623
Conversation
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.
I think touching all of these files is a good opportunity to take advantage of async file I/O. There's a lot of disk reads that are blocking atm
All comments addressed. Also opportunistically cleaned up logging in |
This is another in a series of PRs that modernize our development tasks.
PR highlights:
gulp
file streaming fromserver-tests
,presubmit
,caches-json
,performance-urls
, andget-zindex
bing
to the list of expected caches incaches.json
caches-json
andperformance-urls
For easy reviewing (without whitespace-only changes), use this link: https://github.com/ampproject/amphtml/pull/32623/files?w=1
Partial fix for #32585