Skip to content

chore: upgrade puppeteer, only download chrome in browser test action, turn on yarn cache#646

Merged
szuperaz merged 8 commits intomainfrom
cache-on-ci
Mar 11, 2026
Merged

chore: upgrade puppeteer, only download chrome in browser test action, turn on yarn cache#646
szuperaz merged 8 commits intomainfrom
cache-on-ci

Conversation

@szuperaz
Copy link
Contributor

@szuperaz szuperaz commented Mar 11, 2026

Attempting to fix never-ending CI actions:

  • I'm fairly certain the root cause is Yarn@1 issue of postinstall scripts can hang forever - I'm not sure why this only started recently, but this PR should solve this issue by (maybe installing esbuild, which also has a postinstall, triggered the bug?):
    • Apart from browser test chrome is not installed -> this will make all actions faster
    • In browser chrome is installed in a separate step, not in postinstall
  • Unrelated but
    • Upgraded puppeteer
    • Updated setup-node and checkout actions to use node 24
    • Enabled yarn cache (should be more complete than the previous solution of caching node_modules only)
    • Restore running unit tests on node 16
    • Saved package manager version to package.json

@github-actions
Copy link
Contributor

github-actions bot commented Mar 11, 2026

Size Change: 0 B

Total Size: 109 kB

ℹ️ View Unchanged
Filename Size
dist/js_min/getstream.js 35.5 kB
dist/js/getstream.js 73.9 kB

compressed-size-action

@szuperaz szuperaz changed the title chore: turn on yarn cache chore: upgrade puppeteer, turn on yarn cache Mar 11, 2026
@szuperaz szuperaz changed the title chore: upgrade puppeteer, turn on yarn cache chore: upgrade puppeteer, only download chrome in browser test action, turn on yarn cache Mar 11, 2026
@szuperaz szuperaz merged commit 1196def into main Mar 11, 2026
11 checks passed
@szuperaz szuperaz deleted the cache-on-ci branch March 11, 2026 14:32
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